Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229193317
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/SafeVariableLengthDimensionDataChunkStore.java
---
@@ -169,7 +169,7 @@ public void fillRow(int rowId, CarbonColumnVector
vector, int vectorRow) {
length)) {
vector.putNull(vectorRow);
--- End diff --
Need to skip this for varchar also ?---
