Github user rahulforallp commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2293#discussion_r187533373
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -481,9 +482,13 @@ private void buildTableSchema(Field[] fields,
TableSchemaBuilder tableSchemaBuil
ColumnSchema columnSchema = tableSchemaBuilder
.addColumn(new StructField(field.getFieldName(),
field.getDataType()),
valIndex, isSortColumn > -1);
- columnSchema.setSortColumn(true);
if (isSortColumn > -1) {
+ columnSchema.setSortColumn(true);
sortColumnsSchemaList[isSortColumn] = columnSchema;
+ } else if (sortColumnsList.isEmpty() &&
columnSchema.isDimensionColumn()) {
--- End diff --
complex check is already handled above.
---
