Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r200897566
--- 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)
+ var noInvColIntersecLongStrCols = longStringColumns
+ .intersect(noInvertedIdxCols.map(col => col.toUpperCase))
+ if (!noInvColIntersecLongStrCols.isEmpty) {
+ throw new MalformedCarbonCommandException(
+ "Column(s):" + noInvColIntersecLongStrCols.mkString(",") +
+ " both in no_inverted_index and " +
--- End diff --
Why this is not allowed? Please add comment in this if check
---
