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

    https://github.com/apache/carbondata/pull/2990#discussion_r243248566
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/util/AlterTableUtil.scala ---
    @@ -269,12 +269,50 @@ object AlterTableUtil {
               }
             }
           }
    -      metastore
    +      metaStore
             
.revertTableSchemaInAlterFailure(carbonTable.getCarbonTableIdentifier,
               thriftTable, 
carbonTable.getAbsoluteTableIdentifier)(sparkSession)
         }
       }
     
    +  /**
    +   * This method modifies the table properties if column rename happened
    +   *
    +   * @param tableProperties
    --- End diff --
    
    add comment for all parameter


---

Reply via email to