Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2326#discussion_r189787603
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -406,8 +406,9 @@ private CarbonTable buildCarbonTable() {
ColumnSchema[] sortColumnsSchemaList = new
ColumnSchema[sortColumnsList.size()];
Field[] fields = schema.getFields();
buildTableSchema(fields, tableSchemaBuilder, sortColumnsList,
sortColumnsSchemaList);
-
-
tableSchemaBuilder.setSortColumns(Arrays.asList(sortColumnsSchemaList));
+ if (!(sortColumnsSchemaList.length == 1 && sortColumnsSchemaList[0] ==
null)) {
--- End diff --
this scenario should not handled....in this case throw exception
---
