Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2918
@agresch If this strategy only happens when cluster resource is full
originally, i'm ok with it. At least it's a better way.
---
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2915
@revans2 could you help me to review this PR? thanks
---
Github user agresch commented on the issue:
https://github.com/apache/storm/pull/2918
@revans2 @danny0405 - Made changes requested by @revans2. Let me know if
we should have a separate JIRA for switching blacklisting to work strictly by
supervisor instead of doing this mismatch
Github user agresch commented on the issue:
https://github.com/apache/storm/pull/2918
@revans2 - I will look at addressing DefaultBlacklistStrategy.
---
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239522894
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239517824
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239519351
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239461731
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
