Github user Ethanlm commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2754#discussion_r209066631
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
    @@ -267,8 +267,8 @@
         private static final Histogram numAddedSlotPerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-added-slots-per-scheduling", 
new ExponentiallyDecayingReservoir());
         private static final Histogram numRemovedExecPerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-removed-executors-per-scheduling",
 new ExponentiallyDecayingReservoir());
         private static final Histogram numRemovedSlotPerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-removed-slots-per-scheduling",
 new ExponentiallyDecayingReservoir());
    -    private static final Histogram numNetExecChangePerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-net-executors-changed-per-scheduling",
 new ExponentiallyDecayingReservoir());
    -    private static final Histogram numNetSlotChangePerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-net-slots-changed-per-scheduling",
 new ExponentiallyDecayingReservoir());
    +    private static final Histogram numNetExecIncreasePerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-net-executors-changed-per-scheduling",
 new ExponentiallyDecayingReservoir());
    +    private static final Histogram numNetSlotIncreasePerScheduling = 
StormMetricsRegistry.registerHistogram("nimbus:num-net-slots-changed-per-scheduling",
 new ExponentiallyDecayingReservoir());
    --- End diff --
    
    better to change `nimbus:num-net-slots-changed-per-scheduling` too


---

Reply via email to