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

    https://github.com/apache/storm/pull/2458#discussion_r157074167
  
    --- Diff: conf/storm.yaml.example ---
    @@ -72,4 +72,11 @@
     #     argument:
     #       - endpoint: "metrics-collector.mycompany.org"
     #
    -# storm.cluster.metrics.consumer.publish.interval.secs: 60
    \ No newline at end of file
    +# storm.cluster.metrics.consumer.publish.interval.secs: 60
    +
    +# Event Logger
    +# topology.event.logger.register:
    +#   - class: "org.apache.storm.metric.FileBasedEventLogger"
    --- End diff --
    
    nit: remove the metric package name reference
    



---

Reply via email to