Github user akashrn5 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2422#discussion_r199402754
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchemaCommon.scala
---
@@ -440,7 +544,7 @@ class TableNewProcessor(cm: TableModel) {
}
// varchar column is a string column that in long_string_columns
- private def isVarcharColumn(colName : String): Boolean = {
--- End diff --
same as above, please crosscheck and remove unnnecessary changes
---
