Fix bug in exclude-only case with no doc values in segment

Signed-off-by: Michael Froh <[email protected]>
opensearch-project · Aug 5, 2024 · f6e1c96 · f6e1c96
1 parent d93db7e
commit f6e1c96
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java b/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java
@@ -480,11 +480,11 @@ private static void process(
         @Override
         public LongBitSet acceptedGlobalOrdinals(SortedSetDocValues globalOrdinals) throws IOException {
             LongBitSet accept = new LongBitSet(globalOrdinals.getValueCount());
-            if (includePrefixes.isEmpty()) {
+            if (!includePrefixes.isEmpty()) {
+                process(globalOrdinals, accept.length(), includePrefixes, accept::set);
+            } else if (accept.length() > 0) {
                 // Exclude-only
                 accept.set(0, accept.length());
-            } else {
-                process(globalOrdinals, accept.length(), includePrefixes, accept::set);
             }
             process(globalOrdinals, accept.length(), excludePrefixes, accept::clear);
             return accept;