Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2764#discussion_r208965019
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -2918,6 +2943,10 @@ public void launchServer() throws Exception {
}
});
}
+
+ //Should we make the delaySecs and recurSecs in sync with any
conf value?
+ // They should be around the reporting interval, but it's not
configurable
+ timer.scheduleRecurring(5, 5, clusterMetricSet);
--- End diff --
why do we set to 5 and 5
---
