Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2375#discussion_r196308657
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -289,7 +290,79 @@ abstract class CarbonDDLSqlParser extends
AbstractCarbonSparkSQLParser {
noDictionaryDims, msrs, dims)
if (groupCols != null) {
throw new MalformedCarbonCommandException(
- s"${CarbonCommonConstants.COLUMN_GROUPS} is deprecated")
+ s"${ CarbonCommonConstants.COLUMN_GROUPS } is deprecated")
+ }
+
+ // validate the local dictionary property if defined
+ if
(tableProperties.get(CarbonCommonConstants.LOCAL_DICT_ENABLE).isDefined) {
+ // if any invalid value is configured for LOCAL_DICT_ENABLE, then
default value will be
+ // considered which is true
+ if (!CarbonScalaUtil
+
.castStringToBoolean(tableProperties(CarbonCommonConstants.LOCAL_DICT_ENABLE)))
{
--- End diff --
You can directly use
```
Try(tableProperties(CarbonCommonConstants.LOCAL_DICT_ENABLE).toBoolean)
match {
case Success(value) =>
case Failure(e) =>
}
```
---
