[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_r209088518
  
    --- 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 --
    
    I can think of an Alternative
    
    ```java
    enum ExceptionMeters {
        ENUM1;
    
        private static final Map<...> map = ...;
        private static final MetricSet set = () -> map;
    
        public static MetricSet getMetricSet() {
            return set;
    }
    ```


---