This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 6b123cb965 Upgrade RoaringBitmap to 1.0.5 to pick up the fix for 
RangeBitmap.between() (#12604)
6b123cb965 is described below

commit 6b123cb965cf17e5144b2d85cd36d5c9eb7cb089
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Fri Mar 8 10:50:32 2024 -0800

    Upgrade RoaringBitmap to 1.0.5 to pick up the fix for RangeBitmap.between() 
(#12604)
---
 .../segment/local/segment/index/readers/BitSlicedRangeIndexReader.java | 3 +--
 pom.xml                                                                | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java
 
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java
index a761603d3a..a21891e238 100644
--- 
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java
+++ 
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/readers/BitSlicedRangeIndexReader.java
@@ -188,8 +188,7 @@ public class BitSlicedRangeIndexReader implements 
RangeIndexReader<ImmutableRoar
         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
index 2a894c35d5..98a2fd0041 100644
--- 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>


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to