[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-06 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2400 @jerrypeng yes I need to go follow up on my previous ask and see where the lawyers are at with it. ---

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-06 Thread jerrypeng
Github user jerrypeng commented on the issue: https://github.com/apache/storm/pull/2400 @revans2 yup you are right about it being more about the size of the component. Great work! Thanks for sharing the detailed information as its a good learning experience for me! Keep me in to lo

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-06 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2400 @jerrypeng, I am not sure your intuition is right, but this is all still theoretical until we roll it out and see what happens in real life. We have run some simulations that at least for o

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-06 Thread jerrypeng
Github user jerrypeng commented on the issue: https://github.com/apache/storm/pull/2400 @revans2 thanks for the explanation! Sorry my math was wrong! I understand the context a lot better now! Though my intuition tells me that if larger topologies usually get scheduled first, this wi

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-06 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2400 This is very long and I don't expect everyone to read the whole thing. @jerrypeng your numbers are off for big vs small topologies. The scores will be negative for your example. -0.

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-03 Thread jerrypeng
Github user jerrypeng commented on the issue: https://github.com/apache/storm/pull/2400 @revans2 can you elaborate on why EvictionPolicies are no longer needed? ---

[GitHub] storm issue #2400: STORM-2792: Remove RAS EvictionPolicy and cleanup

2017-11-03 Thread jerrypeng
Github user jerrypeng commented on the issue: https://github.com/apache/storm/pull/2400 @revans2 interesting work! Though the formula used for the DefaultSchedulingPriorityStrategy seems to have a bias towards smaller topologies. For example: Total amount of Resourc