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
---