Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2764#discussion_r209154387 --- Diff: storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java --- @@ -4652,4 +4745,192 @@ public IScheduler getForcedScheduler() { } + //enum NotPorted { + // //Declared in StormConf. I don't see the value in reporting so. + // SUPERVISOR_TOTAL_RESOURCE, + // //May be able to aggregate based on status; + // TOPOLOGY_STATUS, + // TOPOLOGY_SCHED_STATUS, + // //May be aggregated, as well as other distinct values + // NUM_DISTINCT_NIMBUS_VERSION; + //} + + private class ClusterSummaryMetricSet implements MetricSet, Runnable { --- End diff -- Then `isLeader` can be made public or protected, and Nimbus can be injected into this class via a constructor.
---