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

[srdo]

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

    https://github.com/apache/storm/pull/2754#discussion_r209083886
  
    --- 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 --
    
    You're right, but can't a lambda solve this? Would it work to do 
`registerMetricSet(ExceptionMeters::getMetrics)` in line 132 instead?


---