Skip to content

Commit 66324f7

Browse files
committed
Fix test failure.
The test expects that opening a writer on 5 segments doesn't cause merging, but actually it does since randomization created a merge policy with a factor of 5.
1 parent 8fa0de2 commit 66324f7

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

lucene/core/src/test/org/apache/lucene/index/TestDeletionPolicy.java

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -459,7 +459,8 @@ public void testOpenPriorSnapshot() throws IOException {
459459
dir,
460460
newIndexWriterConfig(new MockAnalyzer(random()))
461461
.setIndexDeletionPolicy(policy)
462-
.setIndexCommit(lastCommit));
462+
.setIndexCommit(lastCommit)
463+
.setMergePolicy(newLogMergePolicy(10)));
463464
assertEquals(10, writer.getDocStats().numDocs);
464465

465466
// Should undo our rollback:
@@ -476,12 +477,13 @@ public void testOpenPriorSnapshot() throws IOException {
476477
dir,
477478
newIndexWriterConfig(new MockAnalyzer(random()))
478479
.setIndexDeletionPolicy(policy)
479-
.setIndexCommit(lastCommit));
480+
.setIndexCommit(lastCommit)
481+
.setMergePolicy(newLogMergePolicy(10)));
480482
assertEquals(10, writer.getDocStats().numDocs);
481483
// Commits the rollback:
482484
writer.close();
483485

484-
// Now 8 because we made another commit
486+
// Now 7 because we made another commit
485487
assertEquals(7, DirectoryReader.listCommits(dir).size());
486488

487489
r = DirectoryReader.open(dir);
@@ -507,7 +509,10 @@ public void testOpenPriorSnapshot() throws IOException {
507509
// but this time keeping only the last commit:
508510
writer =
509511
new IndexWriter(
510-
dir, newIndexWriterConfig(new MockAnalyzer(random())).setIndexCommit(lastCommit));
512+
dir,
513+
newIndexWriterConfig(new MockAnalyzer(random()))
514+
.setIndexCommit(lastCommit)
515+
.setMergePolicy(newLogMergePolicy(10)));
511516
assertEquals(10, writer.getDocStats().numDocs);
512517

513518
// Reader still sees fully merged index, because writer

0 commit comments

Comments
 (0)