Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1720#discussion_r158512534
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -1938,6 +1939,12 @@ public static TableInfo
convertGsonToTableInfo(Map<String, String> properties) {
}
builder.append(part);
}
+
+ // Datatype GSON adapter is added to support backward compatibility
for tableInfo desrialization
+ GsonBuilder gsonBuilder = new GsonBuilder();
+ gsonBuilder.registerTypeAdapter(DataType.class,
DataTypes.DataTypeAdapter.getInstance());
--- End diff --
I am not sure whether it has concurrent issue if we use singleton here, I
think it is acceptable to new a DataTypeAdapter
---
