[GitHub] storm pull request #2504: STORM-2887: store metrics into RocksDB

[agresch]

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

    https://github.com/apache/storm/pull/2504#discussion_r160804460
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/supervisor/Slot.java ---
    @@ -936,6 +936,9 @@ static DynamicState handleRunning(DynamicState 
dynamicState, StaticState staticS
                 }
                 dynamicState = dynamicState.withProfileActions(mod, 
modPending);
             }
    +
    +        dynamicState.container.processMetrics();
    --- End diff --
    
    Yes, will fix.


---