Skip to content

Commit

Permalink
Fix Flaky Test ShuffleForcedMergePolicyTests.testDiagnostics
Browse files Browse the repository at this point in the history
Signed-off-by: kkewwei <[email protected]>
Signed-off-by: kkewwei <[email protected]>
  • Loading branch information
kkewwei committed Feb 19, 2025
1 parent e62bf1a commit c56977f
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
import org.apache.lucene.index.LeafReaderContext;
import org.apache.lucene.index.MergePolicy;
import org.apache.lucene.index.SegmentInfos;
import org.apache.lucene.index.TieredMergePolicy;
import org.apache.lucene.search.Sort;
import org.apache.lucene.search.SortField;
import org.apache.lucene.store.Directory;
Expand All @@ -59,7 +60,11 @@ public class ShuffleForcedMergePolicyTests extends BaseMergePolicyTestCase {
public void testDiagnostics() throws IOException {
try (Directory dir = newDirectory()) {
IndexWriterConfig iwc = newIndexWriterConfig().setMaxFullFlushMergeWaitMillis(0);
MergePolicy mp = new ShuffleForcedMergePolicy(newTieredMergePolicy());
TieredMergePolicy tieredMergePolicy = newTieredMergePolicy();
// ensure only trigger one Merge when flushing, and there are remaining segments to be force merged
tieredMergePolicy.setSegmentsPerTier(8);
tieredMergePolicy.setMaxMergeAtOnce(8);
MergePolicy mp = new ShuffleForcedMergePolicy(tieredMergePolicy);
iwc.setMergePolicy(mp);
boolean sorted = random().nextBoolean();
if (sorted) {
Expand Down

0 comments on commit c56977f

Please sign in to comment.