[ 
https://issues.apache.org/jira/browse/FLINK-2997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15197311#comment-15197311
 ] 

ASF GitHub Bot commented on FLINK-2997:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/1776#discussion_r56332311
  
    --- Diff: 
flink-optimizer/src/main/java/org/apache/flink/optimizer/traversals/RangePartitionRewriter.java
 ---
    @@ -221,7 +223,7 @@ public void postVisit(PlanNode node) {
                prRemoverNode.setParallelism(targetParallelism);
                prPlanNode.setParallelism(targetParallelism);
                GlobalProperties globalProperties = new GlobalProperties();
    -           globalProperties.setRangePartitioned(new Ordering(0, null, 
Order.ASCENDING));
    +           globalProperties.setRangePartitioned(new Ordering(0, null, 
Order.ASCENDING), channel.getDataDistribution());
    --- End diff --
    
    can you pass `null` instead of `channel.getDataDistribution()` to make it 
clear that no provided data distribution is available?


> Support range partition with user customized data distribution.
> ---------------------------------------------------------------
>
>                 Key: FLINK-2997
>                 URL: https://issues.apache.org/jira/browse/FLINK-2997
>             Project: Flink
>          Issue Type: New Feature
>            Reporter: Chengxiang Li
>
> This is a followup work of FLINK-7, sometime user have better knowledge of 
> the source data, and they can build customized data distribution to do range 
> partition more efficiently.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to