Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2458#discussion_r157090250
--- 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"
+# - class: "org.mycompany.MyEventLogger"
+# argument:
--- End diff --
I'm following the name for metrics consumer, but I think it is worth to
fix. Moreover we're deprecating metrics consumer so it looks OK. I'll fix it.
---
