Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542948
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -391,6 +391,24 @@ abstract class CarbonDDLSqlParser extends
AbstractCarbonSparkSQLParser {
tableProperties.get(CarbonCommonConstants.CACHE_LEVEL).get,
tableProperties)
}
+ // validate long_string_columns from tale properties
+ var longStringColumns = varcharColumns.map(column =>
column.toUpperCase)
--- End diff --
ok
---