Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2422#discussion_r199568724
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
@@ -278,14 +282,14 @@ class AlterTableColumnSchemaGenerator(
// Its based on the dimension name and measure name
allColumns.filter(x => !x.isInvisible).groupBy(_.getColumnName)
.foreach(f => if (f._2.size > 1) {
--- End diff --
change f => to (columnName, count) => for better readability
---
