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

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


The following commit(s) were added to refs/heads/master by this push:
     new 9f6c20b  skip bit-map inverted index generation for sorted column 
(#4270)
9f6c20b is described below

commit 9f6c20bfe07b4356a5e5fec5deb3f8337eed3c16
Author: JasperJiaGuo <[email protected]>
AuthorDate: Mon Jun 10 05:36:48 2019 -0700

    skip bit-map inverted index generation for sorted column (#4270)
    
    * skip the inverted index generation
    
    * skip the cleaning, no potential side effect
    
    * fixed comments
    
    * fixed comments
    
    * fixed comments
    
    * standalone test done
---
 .../pinot/core/segment/creator/impl/SegmentColumnarIndexCreator.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/SegmentColumnarIndexCreator.java
 
b/pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/SegmentColumnarIndexCreator.java
index 14ba041..2016776 100644
--- 
a/pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/SegmentColumnarIndexCreator.java
+++ 
b/pinot-core/src/main/java/org/apache/pinot/core/segment/creator/impl/SegmentColumnarIndexCreator.java
@@ -157,8 +157,8 @@ public class SegmentColumnarIndexCreator implements 
SegmentCreator {
                   indexCreationInfo.getTotalNumberOfEntries()));
         }
 
-        // Initialize inverted index creator
-        if (invertedIndexColumns.contains(columnName)) {
+        // Initialize inverted index creator; skip creating inverted index if 
sorted
+        if (invertedIndexColumns.contains(columnName) && 
!indexCreationInfo.isSorted()) {
           if (segmentCreationSpec.isOnHeap()) {
             _invertedIndexCreatorMap
                 .put(columnName, new 
OnHeapBitmapInvertedIndexCreator(_indexDir, columnName, cardinality));


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

Reply via email to