Github user kumarvishal09 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2654#discussion_r214088957
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/EncodingFactory.java
 ---
    @@ -81,25 +81,29 @@ public ColumnPageDecoder createDecoder(List<Encoding> 
encodings, List<ByteBuffer
           metadata.readFields(in);
           SimpleStatsResult stats = 
PrimitivePageStatsCollector.newInstance(metadata);
           return new AdaptiveIntegralCodec(metadata.getSchemaDataType(), 
metadata.getStoreDataType(),
    -          stats).createDecoder(metadata);
    +          stats, encodings.contains(Encoding.INVERTED_INDEX),
    --- End diff --
    
    why two times we are calling encodings.contains(Encoding.INVERTED_INDEX) 
same value can't be use 


---

Reply via email to