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

    https://github.com/apache/carbondata/pull/1678#discussion_r157780425
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataRefNodeWrapper.java
 ---
    @@ -124,11 +124,10 @@ public DimensionRawColumnChunk 
getDimensionChunk(FileHolder fileReader, int bloc
       private DimensionColumnChunkReader getDimensionColumnChunkReader() 
throws IOException {
         ColumnarFormatVersion version =
             
ColumnarFormatVersion.valueOf(blockInfos.get(index).getDetailInfo().getVersionNumber());
    -    DimensionColumnChunkReader dimensionColumnChunkReader = 
CarbonDataReaderFactory.getInstance()
    +    return CarbonDataReaderFactory.getInstance()
             .getDimensionColumnChunkReader(version,
    --- End diff --
    
    move .getDimensionColumnChunkReader to previous line


---

Reply via email to