forked from elastic/elasticsearch
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix testPreferCopyWithHighestMatchingOperations
In elastic#74081 this test failed with a `NoNodeAvailableException` within the `indexRandom()` call immediately after stopping a node. This could happen if the `node-left` event wasn't fully applied before calling `indexRandom()` with an empty list of docs but with `forceRefresh` set to true: since there's no docs, the replica wouldn't be marked as stale, so the final refresh would detect the missing node, failing its `assertNoFailures` wrapper. This commit avoids calling `indexRandom()` with no docs in this location. It also enhances `assertNoFailures` to report the details of each failure, rather than just the summary. Closes elastic#74081
- Loading branch information
1 parent
bb37e09
commit 3ed697a
Showing
2 changed files
with
13 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters