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

    https://github.com/apache/carbondata/pull/2252#discussion_r190103945
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/chunk/reader/dimension/v2/CompressedDimensionChunkFileBasedReaderV2.java
 ---
    @@ -121,6 +121,7 @@ public DimensionColumnPage decodeColumnPage(
         int[] invertedIndexesReverse = new int[0];
         int[] rlePage = null;
         DataChunk2 dimensionColumnChunk = null;
    +    boolean isLongStringColumn = 
dimensionRawColumnChunk.isLongStringColumn();
    --- End diff --
    
    The same as above. V2 does not support longStringColumn, so it is fine that 
the `isLongStringColumn` will always be false.


---

Reply via email to