Github user danny0405 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239306840
--- 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/2920#discussion_r239235259
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/NimbusMetricProcessor.java
---
@@ -24,7 +24,7 @@
public void
Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2920#discussion_r239231045
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/NimbusMetricProcessor.java
---
@@ -24,7 +24,7 @@
public void
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2917
Ok, just wanted to be sure that there wasn't an issue to document.
+1
---
GitHub user agresch opened a pull request:
https://github.com/apache/storm/pull/2920
STORM-3297 prevent supervisor restart when no nimbus leader exists
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/agresch/storm
I was wondering if there was support to change
https://git.ouroath.com/storm/storm/blob/master/storm-client/src/jvm/org/apache/storm/utils/NimbusLeaderNotFoundException.java
to a checked exception.
Github user kishorvpatil commented on the issue:
https://github.com/apache/storm/pull/2917
@srdo There was no explicit issue. Just wanted to upgrade to latest on
jetty minor version
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2917
Looks good. Could you update the commit message to contain the issue number?
Is this upgrade fixing a known issue?
---
GitHub user srdo opened a pull request:
https://github.com/apache/storm/pull/2919
STORM-3296: Upgrade curator-test to avoid CURATOR-409
https://issues.apache.org/jira/browse/STORM-3296
You can merge this pull request into a Git repository by running:
$ git pull
Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239066404
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
Github user danny0405 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2918#discussion_r239039664
--- Diff:
storm-server/src/main/java/org/apache/storm/scheduler/blacklist/strategies/RasBlacklistStrategy.java
---
@@ -79,25 +81,46 @@
11 matches
Mail list logo