Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2764#discussion_r209341987
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -2865,16 +2890,14 @@ public void launchServer() throws Exception {
}
});
- StormMetricsRegistry.registerGauge("nimbus:num-supervisors",
() -> state.supervisors(null).size());
- StormMetricsRegistry.registerGauge("nimbus:fragmented-memory",
this::fragmentedMemory);
- StormMetricsRegistry.registerGauge("nimbus:fragmented-cpu",
this::fragmentedCpu);
- StormMetricsRegistry.registerGauge("nimbus:available-memory",
() -> nodeIdToResources.get().values()
+ // Num supervisor, and fragmented resources have been included
in cluster summary
--- End diff --
Please remove, unless you're looking at the diff, you won't know what this
means
---