Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1382#discussion_r141365644
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
@@ -190,6 +190,14 @@ class AlterTableColumnSchemaGenerator(
val LOGGER = LogServiceFactory.getLogService(this.getClass.getName)
+ def checkSortColumn(columnName: String): Boolean = {
+ val sortColumns = alterTableModel.tableProperties.get("sort_columns")
+ if(sortColumns.isDefined) {
+ sortColumns.get.contains(columnName)
+ } else {
+ true
--- End diff --
by default sort column is enabled for all the dimension column, if user is
not providing any column property then it is true
---
