Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2953#discussion_r236543235
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java
---
@@ -536,7 +537,8 @@ public static Dictionary
getDictionary(AbsoluteTableIdentifier absoluteTableIden
*/
public static Map<String, List<Distributable>>
nodeBlockMapping(List<Distributable> blockInfos) {
// -1 if number of nodes has to be decided based on block location
information
- return nodeBlockMapping(blockInfos, -1);
+ return nodeBlockMapping(blockInfos, -1, null,
--- End diff --
the choice of BlockAssignmentStrategy should be judged by
CarbonProperties.getInstance().isLoadSkewedDataOptimizationEnabled() ?
---
