Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2252#discussion_r189912968
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/reader/dimension/v1/CompressedDimensionChunkFileBasedReaderV1.java
---
@@ -99,6 +99,7 @@ public CompressedDimensionChunkFileBasedReaderV1(final
BlockletInfo blockletInfo
@Override public DimensionColumnPage decodeColumnPage(
DimensionRawColumnChunk dimensionRawColumnChunk, int pageNumber)
throws IOException {
+ boolean isLongStringColumn =
dimensionRawColumnChunk.isLongStringColumn();
--- End diff --
In V1 case it will be always false...as new encoder type will be only
supported for V3 format
---
