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

    https://github.com/apache/carbondata/pull/2390#discussion_r197027343
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
 ---
    @@ -400,6 +403,39 @@ abstract class CarbonDDLSqlParser extends 
AbstractCarbonSparkSQLParser {
           tableComment)
       }
     
    +  /**
    +   * this method validates whether any of the child columns of complex 
dataType column
    +   * is of type string.
    +   */
    +  def validateChildColumns(field: Field): Boolean = {
    --- End diff --
    
    can you change the logic to return true as soon as you find any column with 
string datatype. No need to check other columns.


---

Reply via email to