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

    https://github.com/apache/carbondata/pull/2390#discussion_r197019592
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
 ---
    @@ -444,16 +476,30 @@ abstract class CarbonDDLSqlParser extends 
AbstractCarbonSparkSQLParser {
             throw new MalformedCarbonCommandException(errormsg)
           }
         }
    +
    +    // Validate whether any of the child columns of complex dataType 
column is a string column
    +    localDictColumns.foreach { dictColm =>
    +      if (fields
    +        .exists(x => x.column.equalsIgnoreCase(dictColm) && 
x.children.isDefined &&
    +                     null != x.children.get &&
    +                     !validateChildColumns(x))) {
    +        val errMsg = "No child column is string dataType column in 
local_dictionary_include."
    --- End diff --
    
    error message is not correct, give a proper error message


---

Reply via email to