[
https://issues.apache.org/jira/browse/FLINK-10820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16683334#comment-16683334
]
ASF GitHub Bot commented on FLINK-10820:
----------------------------------------
zhijiangW commented on issue #7051: [FLINK-10820][network] Simplify the
RebalancePartitioner implementation
URL: https://github.com/apache/flink/pull/7051#issuecomment-437784191
Thanks for reviews, @Clarkkkkk @pnowojski .
I have updated the codes for addressing the above comments!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Simplify the RebalancePartitioner implementation
> ------------------------------------------------
>
> Key: FLINK-10820
> URL: https://issues.apache.org/jira/browse/FLINK-10820
> Project: Flink
> Issue Type: Sub-task
> Components: Network
> Affects Versions: 1.8.0
> Reporter: zhijiang
> Assignee: zhijiang
> Priority: Minor
> Labels: pull-request-available
>
> The current {{RebalancePartitioner}} implementations seems a little hacky for
> selecting a random number as the first channel index, and the following
> selections based on this random index in round-robin fashion.
> We can define a constant as the first channel index to make the
> implementation simple and readable. To do so, it will not change the
> rebalance semantics.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
