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 727d0c9cf1 range index shouldn’t be loaded when the column is sorted 
(#8571)
727d0c9cf1 is described below

commit 727d0c9cf1babbd7d062c32945517629ba504c39
Author: Xiaobing <61892277+klsi...@users.noreply.github.com>
AuthorDate: Wed Apr 20 12:19:41 2022 -0700

    range index shouldn’t be loaded when the column is sorted (#8571)
---
 .../local/segment/index/column/PhysicalColumnIndexContainer.java        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/column/PhysicalColumnIndexContainer.java
 
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/column/PhysicalColumnIndexContainer.java
index 139a114856..17f58571c5 100644
--- 
a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/column/PhysicalColumnIndexContainer.java
+++ 
b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/column/PhysicalColumnIndexContainer.java
@@ -123,7 +123,7 @@ public final class PhysicalColumnIndexContainer implements 
ColumnIndexContainer
       _bloomFilter = null;
     }
 
-    if (loadRangeIndex) {
+    if (loadRangeIndex && !metadata.isSorted()) {
       PinotDataBuffer buffer = segmentReader.getIndexFor(columnName, 
ColumnIndexType.RANGE_INDEX);
       _rangeIndex = indexReaderProvider.newRangeIndexReader(buffer, metadata);
     } else {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to