Skip to content

Remove shard multiple data path selections.#75822

Merged
rjernst merged 3 commits intoelastic:masterfrom
TencentCloudES:clean_multi_paths
Jul 30, 2021
Merged

Remove shard multiple data path selections.#75822
rjernst merged 3 commits intoelastic:masterfrom
TencentCloudES:clean_multi_paths

Conversation

@howardhuanghua
Copy link
Copy Markdown
Contributor

Clean up mutiple data path selections for new shard creation.
Relates #71205.

@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Jul 29, 2021
@mark-vieira mark-vieira added the :Core/Infra/Core Core issues without another label label Jul 29, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jul 29, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@rjernst rjernst self-requested a review July 29, 2021 19:21
@rjernst
Copy link
Copy Markdown
Member

rjernst commented Jul 29, 2021

@elasticmachine ok to test

@rjernst rjernst self-assigned this Jul 29, 2021
@rjernst
Copy link
Copy Markdown
Member

rjernst commented Jul 29, 2021

Thanks for the PR @howardhuanghua, I missed this in my previous work in MDP removal.

It looks like checkstyle is failing due to unused imports. Could you please fix that and I will then be happy to merge once tests pass.

@howardhuanghua
Copy link
Copy Markdown
Contributor Author

@rjernst Thanks for checking. I have updated the branch.

Copy link
Copy Markdown
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @howardhuanghua

@rjernst rjernst merged commit 4d8d88b into elastic:master Jul 30, 2021
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Sep 28, 2021
rjernst added a commit that referenced this pull request Sep 29, 2021
@rjernst rjernst mentioned this pull request Sep 30, 2021
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Core/Infra Meta label for core/infra team v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants