Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2605#discussion_r207280921
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -306,9 +306,12 @@ abstract class CarbonDDLSqlParser extends
AbstractCarbonSparkSQLParser {
CarbonCommonConstants.LOCAL_DICTIONARY_ENABLE_DEFAULT)
}
} else if (!isAlterFlow) {
- // if LOCAL_DICTIONARY_ENABLE is not defined, consider the default
value which is true
- tableProperties.put(CarbonCommonConstants.LOCAL_DICTIONARY_ENABLE,
- CarbonCommonConstants.LOCAL_DICTIONARY_ENABLE_DEFAULT)
+ // if LOCAL_DICTIONARY_ENABLE is not defined, tru to get from system
level property
--- End diff --
spell check "tyu"
---
