[GitHub] storm issue #1874: STORM-2286 Storm Rebalance command should support arbitra...

2017-01-17 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/1874 @arunmahadevan thx for your nice review, i have checked the code The fact is that, now for storm, we will pass in a `:scratch-topology-id` to execute mk-assignments function for rebalance

[GitHub] storm issue #1874: STORM-2286 Storm Rebalance command should support arbitra...

2017-01-17 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/1874 @arunmahadevan There are over 2500+ topologies on our cluster, in most cases, we wanna increase component executors too when we add workers to a topology in order to make full use of these

[GitHub] storm issue #1874: STORM-2286 Storm Rebalance command should support arbitra...

2017-01-16 Thread erikdw
Github user erikdw commented on the issue: https://github.com/apache/storm/pull/1874 I'm unable to reconcile the statements about the ratio always being 1:1 with the content of your "For example" paragraph. Those examples sound like you have having a different number of executors

[GitHub] storm issue #1874: STORM-2286 Storm Rebalance command should support arbitra...

2017-01-16 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/1874 @erikdw thx for your nice review, i have fixed the headline and ticket title. For this PR, the ratio is still 1:1 for tasks and executors for a component, but not for the Storm before