Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2261#discussion_r186049025
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -367,9 +384,8 @@ private CarbonTable buildCarbonTable() {
// user passed size 4 but supplied only 2 fileds
for (Field field : schema.getFields()) {
if (null != field) {
- if (field.getDataType() == DataTypes.STRING ||
- field.getDataType() == DataTypes.DATE ||
- field.getDataType() == DataTypes.TIMESTAMP) {
+ if (field.getDataType() == DataTypes.STRING ||
field.getDataType() == DataTypes.DATE
--- End diff --
Sort column is also supported for long, short, int data type , add a
reverse check if data type is not of double, bigdecimal, struct, array then
add to sort column
---
