Avoid size=0 for terminate_after tests

Signed-off-by: Jay Deng <[email protected]>
jed326 · Oct 6, 2023 · 34b21af · 34b21af
1 parent dad525a
commit 34b21af
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/server/src/internalClusterTest/java/org/opensearch/search/simple/SimpleSearchIT.java b/server/src/internalClusterTest/java/org/opensearch/search/simple/SimpleSearchIT.java
@@ -301,7 +301,8 @@ public void testSimpleTerminateAfterCount() throws Exception {
         SearchResponse searchResponse;
         int size;
         for (int i = 1; i < max; i++) {
-            size = randomIntBetween(0, max);
+            // Size=0 has known issues, see https://github.com/opensearch-project/OpenSearch/issues/10435
+            size = randomIntBetween(1, max);
             searchResponse = client().prepareSearch("test")
                 .setQuery(QueryBuilders.rangeQuery("field").gte(1).lte(max))
                 .setTerminateAfter(i)
@@ -337,8 +338,9 @@ public void testSimpleTerminateAfterTrackTotalHitsUpTo() throws Exception {
         ensureGreen();
         refresh();
 
-        // size=0 is a special case where topDocsCollector is not added
-        int size = randomIntBetween(0, 1);
+        // Size=0 is a special case where topDocsCollector is not added
+        // Size=0 has known issues, see https://github.com/opensearch-project/OpenSearch/issues/10435
+        int size = 1;
         SearchResponse searchResponse;
         searchResponse = client().prepareSearch("test")
             .setQuery(QueryBuilders.rangeQuery("field").gte(1).lte(numDocs))