Upgrade RoaringBitmap to 1.0.5 to pick up the fix for RangeBitmap.bet…

…ween() (#12604)
apache · Mar 8, 2024 · 6b123cb · 6b123cb
1 parent 7fa8bec
commit 6b123cb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
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,8 +188,7 @@ private ImmutableRoaringBitmap queryRangeBitmap(long min, long max, long columnM
         if (min == max) {
           return rangeBitmap.eq(min).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.between(min, max).toMutableRoaringBitmap();
       }
       return rangeBitmap.lte(max).toMutableRoaringBitmap();
     } else {

diff --git a/pom.xml b/pom.xml
@@ -415,7 +415,7 @@
       <dependency>
         <groupId>org.roaringbitmap</groupId>
         <artifactId>RoaringBitmap</artifactId>
-        <version>0.9.38</version>
+        <version>1.0.5</version>
       </dependency>
       <dependency>
         <groupId>com.101tec</groupId>