Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3027#discussion_r244271907 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableColRenameDataTypeChangeCommand.scala --- @@ -262,13 +263,26 @@ private[sql] case class CarbonAlterTableColRenameDataTypeChangeCommand( carbonTable: CarbonTable, tableInfo: TableInfo, addColumnSchema: ColumnSchema, - schemaEvolutionEntry: SchemaEvolutionEntry): Unit = { + schemaEvolutionEntry: SchemaEvolutionEntry, + oldCarbonColumn: CarbonColumn): Unit = { val schemaConverter = new ThriftWrapperSchemaConverterImpl - val a = List(schemaConverter.fromExternalToWrapperColumnSchema(addColumnSchema)) + // get the carbon column in schema order + val carbonColumns = carbonTable.getCreateOrderColumn(carbonTable.getTableName).asScala + .filter(!_.isInvisible).collect{case carbonColumn => carbonColumn.getColumnSchema} --- End diff -- Move filter operation to collect
---