Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1583#discussion_r154984440
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
@@ -320,13 +322,8 @@ class AlterTableColumnSchemaGenerator(
// TODO: move this to carbon store API
object TableNewProcessor {
def apply(
- cm: TableModel,
- identifier: AbsoluteTableIdentifier): TableInfo = {
- new TableNewProcessor(
- cm,
- identifier.getDatabaseName,
- identifier.getTableName,
- identifier.getTablePath).process
+ cm: TableModel): TableInfo = {
--- End diff --
Do not change this, otherwise tablePath will be lost
---