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

apache · Mar 7, 2024 · 4e4f966 · 4e4f966
1 parent f06d5a8
commit 4e4f966
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 {