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

    https://github.com/apache/carbondata/pull/1777#discussion_r161362592
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala
 ---
    @@ -200,7 +200,8 @@ class CarbonHelperSqlAstBuilder(conf: SQLConf,
              throw new MalformedCarbonCommandException("Error: Invalid 
partition definition")
           }
           // partition columns should not be part of the schema
    -      val badPartCols = 
partitionFields.map(_.partitionColumn).toSet.intersect(colNames.toSet)
    +      val badPartCols = partitionFields.map(_.partitionColumn.toLowerCase).
    --- End diff --
    
    please  move `.` to next line, like:
    ```
          val badPartCols = partitionFields
            .map(_.partitionColumn.toLowerCase)
            .toSet
            .intersect(colNames.map(_.toLowerCase).toSet)
    ```


---

Reply via email to