[GitHub] storm pull request #2764: STORM-3147: Port ClusterSummary as metrics to Stor...

[zd-project]

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

    https://github.com/apache/storm/pull/2764#discussion_r208743919
  
    --- Diff: 
storm-server/src/main/java/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java
 ---
    @@ -52,6 +52,7 @@ public void prepare(MetricRegistry metricsRegistry, 
Map<String, Object> topoConf
         public void start() {
             if (reporter != null) {
                 LOG.debug("Starting...");
    +            //TODO: will we make the period customizable?
    --- End diff --
    
    Will log in Jira.


---