Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3046#discussion_r244708432
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java ---
@@ -1491,6 +1491,16 @@ private void validateSortMemorySpillPercentage() {
}
}
+ public int getMaxDictionaryThreshold() {
+ int localDictionaryMaxThreshold = Integer.parseInt(carbonProperties
--- End diff --
add min max validation---
