honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-17 10:25:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix InternalEngineTests#testPruneAwayDeletedButRetainedIds

Browse Source 
The test failed because we had only a single document in the index
that got deleted such that some assertions that expected at least
one live doc failed.

Relates to: #40741
This commit is contained in:
Simon Willnauer 2019-06-06 14:14:41 +02:00
parent 9f7be70f7a
commit 2582e1e8ad
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
server/src/test/java/org/elasticsearch/index/engine/InternalEngineTests.java
View File

@ -5810,6 +5810,7 @@ public class InternalEngineTests extends EngineTestCase {
policy.setMinMergeDocs(10000); policy.setMinMergeDocs(10000);
try (InternalEngine engine = createEngine(indexSettings, store, createTempDir(), policy)) { try (InternalEngine engine = createEngine(indexSettings, store, createTempDir(), policy)) {
int numDocs = between(1, 20); int numDocs = between(1, 20);
logger.info("" + numDocs);
for (int i = 0; i < numDocs; i++) { for (int i = 0; i < numDocs; i++) {
index(engine, i); index(engine, i);
} }
@ -5842,6 +5843,10 @@ public class InternalEngineTests extends EngineTestCase {
assertEquals("the delete and the tombstone", 2, leafReader.numDeletedDocs()); assertEquals("the delete and the tombstone", 2, leafReader.numDeletedDocs());
assertEquals(numDocs + 1, leafReader.maxDoc()); assertEquals(numDocs + 1, leafReader.maxDoc());
Terms id = leafReader.terms("_id"); Terms id = leafReader.terms("_id");
if (numDocs == 1) {
assertNull(id); // everything is pruned away
assertEquals(0, leafReader.numDocs());
} else {
assertNotNull(id); assertNotNull(id);
assertEquals("deleted IDs are pruned away", reader.numDocs(), id.size()); assertEquals("deleted IDs are pruned away", reader.numDocs(), id.size());
TermsEnum iterator = id.iterator(); TermsEnum iterator = id.iterator();
@ -5849,4 +5854,5 @@ public class InternalEngineTests extends EngineTestCase {
} }
} }
} }
}
} }