EngineTestCase#getDocIds should use internal reader (#49564)

dnhatn · dnhatn · commit 244fbf23a92f · 2019-11-25T21:08:08.000-05:00
We do not guarantee that EngineTestCase#getDocIds is called after the engine has been externally refreshed. Hence, we trip an assertion assertSearcherIsWarmedUp. CI: https://gradle-enterprise.elastic.co/s/pm2at5qmfm2iu Relates #48605
diff --git a/test/framework/src/main/java/org/elasticsearch/index/engine/EngineTestCase.java b/test/framework/src/main/java/org/elasticsearch/index/engine/EngineTestCase.java
@@ -1011,7 +1011,7 @@ public static List<DocIdSeqNoAndSource> getDocIds(Engine engine, boolean refresh
         if (refresh) {
             engine.refresh("test_get_doc_ids");
         }
-        try (Engine.Searcher searcher = engine.acquireSearcher("test_get_doc_ids")) {
+        try (Engine.Searcher searcher = engine.acquireSearcher("test_get_doc_ids", Engine.SearcherScope.INTERNAL)) {
             List<DocIdSeqNoAndSource> docs = new ArrayList<>();
             for (LeafReaderContext leafContext : searcher.getIndexReader().leaves()) {
                 LeafReader reader = leafContext.reader();

Original file line number	Diff line number	Diff line change
`@@ -1011,7 +1011,7 @@ public static List<DocIdSeqNoAndSource> getDocIds(Engine engine, boolean refresh`
`1011`	`1011`	`if (refresh) {`
`1012`	`1012`	`engine.refresh("test_get_doc_ids");`
`1013`	`1013`	`}`
`1014`		`- try (Engine.Searcher searcher = engine.acquireSearcher("test_get_doc_ids")) {`
	`1014`	`+ try (Engine.Searcher searcher = engine.acquireSearcher("test_get_doc_ids", Engine.SearcherScope.INTERNAL)) {`
`1015`	`1015`	`List<DocIdSeqNoAndSource> docs = new ArrayList<>();`
`1016`	`1016`	`for (LeafReaderContext leafContext : searcher.getIndexReader().leaves()) {`
`1017`	`1017`	`LeafReader reader = leafContext.reader();`