Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2692#discussion_r215628606
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/model/LoadOption.java
---
@@ -59,7 +59,8 @@
optionsFinal.put("fileheader", Maps.getOrDefault(options,
"fileheader", ""));
optionsFinal.put("commentchar", Maps.getOrDefault(options,
"commentchar", "#"));
optionsFinal.put("columndict", Maps.getOrDefault(options,
"columndict", null));
-
+ optionsFinal.put("sort_scope",
+ Maps.getOrDefault(options, "sort_scope",
CarbonCommonConstants.LOAD_SORT_SCOPE_DEFAULT));
--- End diff --
yes, it should not be set here.
now moved to
CarbonLoadModelBuilder.build() , so it will set only for SDK and
spark-file-format
---