use gte(let) to replace between() which has a bug

apache · Mar 7, 2024 · f35159b · f35159b
1 parent 797e39c
commit f35159b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/.../java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java b/.../java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java
@@ -188,7 +188,8 @@ private ImmutableRoaringBitmap queryRangeBitmap(long min, long max, long columnM
         if (min == max) {
           return rangeBitmap.eq(min).toMutableRoaringBitmap();
         }
-        return rangeBitmap.between(min, max).toMutableRoaringBitmap();
+        // TODO: found bug in between() and use gte(lte) as a workaround for now.
+        return rangeBitmap.gte(min, rangeBitmap.lte(max)).toMutableRoaringBitmap();
       }
       return rangeBitmap.lte(max).toMutableRoaringBitmap();
     } else {