From 6b123cb965cf17e5144b2d85cd36d5c9eb7cb089 Mon Sep 17 00:00:00 2001
From: "Xiaotian (Jackie) Jiang"
 <17555551+Jackie-Jiang@users.noreply.github.com>
Date: Fri, 8 Mar 2024 10:50:32 -0800
Subject: [PATCH] Upgrade RoaringBitmap to 1.0.5 to pick up the fix for
 RangeBitmap.between() (#12604)

---
 .../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 2a894c35d55..98a2fd0041a 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>