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


---

Reply via email to