[GitHub] storm pull request #2754: STORM-3133: Extend metrics on Nimbus and LogViewer

[zd-project]

Github user zd-project commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2754#discussion_r209082396
  
    --- Diff: 
storm-webapp/src/main/java/org/apache/storm/daemon/logviewer/LogviewerServer.java
 ---
    @@ -52,7 +55,7 @@
     /**
      * The main entry of Logviewer.
      */
    -public class LogviewerServer implements AutoCloseable {
    +public class LogviewerServer implements AutoCloseable, MetricSet {
    --- End diff --
    
    It would be weirder since `getMetrics` is an instance method. So I have to 
do `[SOME_ENUM].getMetrics()`


---