This is an automated email from the ASF dual-hosted git repository.
xbli 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 4e4f9666f9 use gte(let) to replace between() which has a bug (#12595)
4e4f9666f9 is described below
commit 4e4f9666f9d8c4d50cbe51b41abadc9534790342
Author: Xiaobing <[email protected]>
AuthorDate: Thu Mar 7 13:15:17 2024 -0800
use gte(let) to replace between() which has a bug (#12595)
---
.../segment/local/segment/index/readers/BitSlicedRangeIndexReader.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
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 a21891e238..a761603d3a 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,7 +188,8 @@ public class BitSlicedRangeIndexReader implements
RangeIndexReader<ImmutableRoar
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 {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
