Github user gvramana commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2299#discussion_r187843922
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/DataTypeUtil.java ---
@@ -401,6 +401,11 @@ public static Object
getDataDataTypeForNoDictionaryColumn(String dimensionValue,
} else if (actualDataType == DataTypes.LONG) {
return ByteUtil.toBytes((Long) dimensionValue);
} else if (actualDataType == DataTypes.TIMESTAMP) {
+ if (dimensionValue instanceof String) {
--- End diff --
Avro supports timestamp as logical type. So need not support this, as avro
supports specific data type
---
