Skip to content

Ensure relocation occur in testRelocationWithConcurrentIndexing#40801

Merged
dnhatn merged 4 commits intoelastic:masterfrom
dnhatn:ensure-shard-allocated
Apr 7, 2019
Merged

Ensure relocation occur in testRelocationWithConcurrentIndexing#40801
dnhatn merged 4 commits intoelastic:masterfrom
dnhatn:ensure-shard-allocated

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Apr 3, 2019

If the relocation is throttled, the subsequent search request on the target node (i.e., with preference _only_nodes=target_node) will fail because some shards have not moved to that node yet. With this change, we will wait for the relocation happens by busily checking the routing table of the testing index on the target node.

Closes #34950

@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests :Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. v7.0.0 v8.0.0 v7.2.0 v6.7.2 labels Apr 3, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

@dnhatn dnhatn requested a review from henningandersen April 3, 2019 16:49
Copy link
Copy Markdown
Contributor

@henningandersen henningandersen 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 @dnhatn , I left one optional comment to consider.

@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Apr 7, 2019

Thanks @henningandersen

@dnhatn dnhatn merged commit 59e9721 into elastic:master Apr 7, 2019
@dnhatn dnhatn deleted the ensure-shard-allocated branch April 7, 2019 17:32
dnhatn added a commit that referenced this pull request Apr 9, 2019
If the relocation is throttled, the subsequent search request on the
target node (i.e., with preference _only_nodes=target_node) will fail
because some shards have not moved to that node yet. With this change,
we will wait for the relocation happens by busily checking the routing
table of the testing index on the target node.

Closes #34950
@dnhatn dnhatn added v7.0.1 and removed v7.0.0 labels Apr 10, 2019
dnhatn added a commit that referenced this pull request Apr 11, 2019
If the relocation is throttled, the subsequent search request on the
target node (i.e., with preference _only_nodes=target_node) will fail
because some shards have not moved to that node yet. With this change,
we will wait for the relocation happens by busily checking the routing
table of the testing index on the target node.

Closes #34950
dnhatn added a commit that referenced this pull request Apr 11, 2019
If the relocation is throttled, the subsequent search request on the
target node (i.e., with preference _only_nodes=target_node) will fail
because some shards have not moved to that node yet. With this change,
we will wait for the relocation happens by busily checking the routing
table of the testing index on the target node.

Closes #34950
gurkankaymak pushed a commit to gurkankaymak/elasticsearch that referenced this pull request May 27, 2019
…tic#40801)

If the relocation is throttled, the subsequent search request on the
target node (i.e., with preference _only_nodes=target_node) will fail
because some shards have not moved to that node yet. With this change,
we will wait for the relocation happens by busily checking the routing
table of the testing index on the target node.

Closes elastic#34950
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Recovery Anything around constructing a new shard, either from a local or a remote source. >test Issues or PRs that are addressing/adding tests v6.7.2 v7.0.1 v7.2.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] testRelocationWithConcurrentIndexing fails

4 participants