Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1679#discussion_r157746071
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/DataTypeUtil.java ---
@@ -353,12 +353,24 @@ public static Object
getDataBasedOnDataTypeForNoDictionaryColumn(byte[] dataInBy
} else if (actualDataType == DataTypes.STRING) {
return
getDataTypeConverter().convertFromByteToUTF8String(dataInBytes);
} else if (actualDataType == DataTypes.SHORT) {
+ if (isEmptyByteArray(dataInBytes)) {
--- End diff --
Please add a comment .. for non string type no dictionary column empty byte
array is null value so no need to parse
---
