Github user akashrn5 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2422#discussion_r199401982
  
    --- 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) {
    -      val name = f._1
    -      LOGGER.error(s"Duplicate column found with name: $name")
    -      LOGGER.audit(
    -        s"Validation failed for Create/Alter Table Operation " +
    -        s"for ${ dbName }.${ alterTableModel.tableName }. " +
    -        s"Duplicate column found with name: $name")
    -      sys.error(s"Duplicate column found with name: $name")
    -    })
    +        val name = f._1
    --- End diff --
    
    correct the style


---

Reply via email to