[
https://issues.apache.org/jira/browse/FLINK-2284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17336916#comment-17336916
]
Flink Jira Bot commented on FLINK-2284:
---------------------------------------
This issue was labeled "stale-major" 7 ago and has not received any updates so
it is being deprioritized. If this ticket is actually Major, please raise the
priority and ask a committer to assign you the issue or revive the public
discussion.
> Confusing/inconsistent PartitioningStrategy
> -------------------------------------------
>
> Key: FLINK-2284
> URL: https://issues.apache.org/jira/browse/FLINK-2284
> Project: Flink
> Issue Type: Bug
> Components: API / DataStream
> Affects Versions: 0.9
> Reporter: Stephan Ewen
> Priority: Major
> Labels: stale-major
>
> The PartitioningStrategy in
> {{org.apache.flink.streaming.runtime.partitioner.StreamPartitioner.java}} is
> non standard and not easily understandable.
> What form of partitioning is `SHUFFLE`? Shuffle just means "redistribute", it
> says nothing about what it does. Same with `DISTRIBUTE`. Also `GLOBAL` is not
> a well-defined/established term. Why is `GROUPBY` a partition type? Doesn't
> grouping simply hash partition (like I assume "SHUFFLE" means), so why does
> it have an extra entry?
> Sticking with principled and established names/concepts is important to allow
> people to collaborate on the code.
> Why not stick with the partitioning types defined in the batch API? They are
> well defined and named:
> ```
> NONE, FORWARD, RANDOM, HASH, RANGE, FORCED_REBALANCE, BROADCAST, CUSTOM
> ```
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
