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

    https://github.com/apache/carbondata/pull/2103#discussion_r179950228
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableAddColumnCommand.scala
 ---
    @@ -85,11 +85,14 @@ private[sql] case class 
CarbonAlterTableAddColumnCommand(
           schemaEvolutionEntry.setAdded(newCols.toList.asJava)
           val thriftTable = schemaConverter
             .fromWrapperToExternalTableInfo(wrapperTableInfo, dbName, 
tableName)
    -      AlterTableUtil
    +      val alterTableVo = AlterTableUtil
    --- End diff --
    
    Better to return tuple instead of return alterTableVo


---

Reply via email to