honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix doc stats test when deleting all docs 

This commit fixes an issue with IndexShardTests#testDocStats when the
number of deleted docs is equal to the number of docs. In this case,
Luence will remove the underlying segment tripping an assertion on the
number of deleted docs.
This commit is contained in:
Jason Tedor 2016-12-23 15:20:42 -05:00
parent 6deb5283db
commit d5c18bf5c9
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
core/src/test/java/org/elasticsearch/index/shard/IndexShardTests.java
View File

@ -1401,7 +1401,8 @@ public class IndexShardTests extends IndexShardTestCase {
IntStream.range(0, Math.toIntExact(numDocs)).boxed().collect(Collectors.toList())); IntStream.range(0, Math.toIntExact(numDocs)).boxed().collect(Collectors.toList()));
for (final Integer i : ids) { for (final Integer i : ids) {
final String id = Integer.toString(i); final String id = Integer.toString(i);
final ParsedDocument doc = testParsedDocument(id, id, "test", null, new ParseContext.Document(), new BytesArray("{}"), null); final ParsedDocument doc =
testParsedDocument(id, id, "test", null, new ParseContext.Document(), new BytesArray("{}"), null);
final Engine.Index index = final Engine.Index index =
new Engine.Index( new Engine.Index(
new Term("_uid", id), new Term("_uid", id),
@ -1428,7 +1429,8 @@ public class IndexShardTests extends IndexShardTestCase {
{ {
final DocsStats docStats = indexShard.docStats(); final DocsStats docStats = indexShard.docStats();
assertThat(docStats.getCount(), equalTo(numDocs)); assertThat(docStats.getCount(), equalTo(numDocs));
assertThat(docStats.getDeleted(), equalTo(numDocsToDelete)); // Lucene will delete a segment if all docs are deleted from it; this means that we lose the deletes when deleting all docs
assertThat(docStats.getDeleted(), equalTo(numDocsToDelete == numDocs ? 0 : numDocsToDelete));
} }
// merge them away // merge them away