Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2692#discussion_r214951956
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -265,12 +262,20 @@ public CarbonWriterBuilder
withLoadOptions(Map<String, String> options) {
!option.equalsIgnoreCase("complex_delimiter_level_1") &&
!option.equalsIgnoreCase("complex_delimiter_level_2") &&
!option.equalsIgnoreCase("quotechar") &&
- !option.equalsIgnoreCase("escapechar")) {
- throw new IllegalArgumentException("Unsupported options. "
- + "Refer method header or documentation");
+ !option.equalsIgnoreCase("escapechar") &&
+ !option.equalsIgnoreCase("sort_scope")) {
+ throw new IllegalArgumentException("Unsupported option:" + option
+ + ". Refer method header or documentation");
}
}
+ // validate sort scope
+ String sortScope = options.get("sort_scope");
+ if ((sortScope != null) &&
(!(sortScope.equalsIgnoreCase("local_sort")) && !(sortScope
--- End diff --
OK. I use this now.
---
