Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2458#discussion_r157090722
--- 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 --
Eventlogging classes are in org.apache.storm.metric package. I don't know
why, but if we can't move them to another package, I think it should be this
value since this requires full class name.
---
