[GitHub] storm issue #2477: STORM-2863: Reset NormalizedResources for tests

2017-12-21 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2477 Thanks, +1 assuming TestGenericResourceAwareStrategy doesn't touch the static fields. ---

[GitHub] storm issue #2477: STORM-2863: Reset NormalizedResources for tests

2017-12-21 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2477 @srdo I moved it over to being a Rule annotation ---

[GitHub] storm issue #2477: STORM-2863: Reset NormalizedResources for tests

2017-12-21 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2477 Does

[GitHub] storm issue #2477: STORM-2863: Reset NormalizedResources for tests

2017-12-21 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2477 @revans2 Could you take a look at https://github.com/apache/storm/pull/2463? I think it will resolve this issue too. ---