Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1581#discussion_r154836902
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/SafeVariableLengthDimensionDataChunkStore.java
---
@@ -141,24 +135,25 @@ public
SafeVariableLengthDimensionDataChunkStore(boolean isInvertedIndex, int nu
// for last record
length = (short) (this.data.length - currentDataOffset);
}
- DataType dt = vector.getType();
- if ((!(dt instanceof StringType) && length == 0) ||
ByteUtil.UnsafeComparer.INSTANCE
+ org.apache.carbondata.core.metadata.datatype.DataType dt =
vector.getType();
--- End diff --
Import this class
---
