From 114c6cd9432714ecefb8716f7cdb223e3f4c0bc3 Mon Sep 17 00:00:00 2001
From: "Xiaotian (Jackie) Jiang" <jackie.jxt@gmail.com>
Date: Fri, 8 Mar 2024 01:11:16 -0800
Subject: [PATCH] Upgrade RoaringBitmap to 1.0.5 to pick up the fix for
 RangeBitmap.between()

---
 .../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 a761603d3a3..a21891e2386 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 @@ 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
index ec7f86ac806..fc108f9b556 100644
--- a/pom.xml
+++ b/pom.xml
@@ -414,7 +414,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>