Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2764#discussion_r209333494
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -2905,6 +2929,10 @@ public void launchServer() throws Exception {
throw new RuntimeException(e);
}
});
+
+ //Should we make the delaySecs and recurSecs in sync with any
conf value?
--- End diff --
Why not update the metricset active flag when Nimbus becomes active instead
of polling for changes?
---
