Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3014#discussion_r245879143
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/SessionParams.java ---
@@ -229,6 +229,12 @@ private boolean validateKeyValue(String key, String
value) throws InvalidConfigu
if (!isValid) {
throw new InvalidConfigurationException("Invalid value " +
value + " for key " + key);
}
+ } else if
(key.startsWith(CarbonLoadOptionConstants.CARBON_TABLE_LOAD_SORT_SCOPE)) {
+ isValid = CarbonUtil.isValidSortOption(value);
+ if (!isValid) {
+ throw new InvalidConfigurationException("The sort scope " + key
+ + " can have only either BATCH_SORT or LOCAL_SORT or
NO_SORT.");
--- End diff --
Sort scope can be global as well. Add in exception message
---
