Github user geetikagupta16 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1601#discussion_r155735141
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala
---
@@ -190,7 +191,7 @@ class CarbonHelperSqlAstBuilder(conf: SQLConf, parser:
CarbonSpark2SqlParser)
}
val tableProperties = mutable.Map[String, String]()
- properties.foreach{property => tableProperties.put(property._1,
property._2)}
+ validatedProperties.foreach{property =>
tableProperties.put(property._1, property._2)}
--- End diff --
Removing the mutable Map will lead to changes in the parameters of other
methods. Should I work on that?
---
