Skip to content

Commit ad6e6ff

Browse files
committed
TEST: Ensure a manual flush always occur
The test testLookupSeqNoByIdInLucene fails because it expects if any change should be visible after a flush. However, that flush might be ignored if the waitIfOngoing parameter is false (the default value), and there is an ongoing flush triggered after merge is running. Closes elastic#35823
1 parent a6ff9f7 commit ad6e6ff

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

server/src/test/java/org/elasticsearch/index/engine/InternalEngineTests.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3809,7 +3809,6 @@ public void testSequenceIDs() throws Exception {
38093809
searchResult.close();
38103810
}
38113811

3812-
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/35823")
38133812
public void testLookupSeqNoByIdInLucene() throws Exception {
38143813
int numOps = between(10, 100);
38153814
long seqNo = 0;
@@ -3878,7 +3877,7 @@ public void testLookupSeqNoByIdInLucene() throws Exception {
38783877
lookupAndCheck.run();
38793878
}
38803879
if (rarely()) {
3881-
engine.flush();
3880+
engine.flush(false, true);
38823881
lookupAndCheck.run();
38833882
}
38843883
}

0 commit comments

Comments
 (0)