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

    https://github.com/apache/storm/pull/2457#discussion_r157072382
  
    --- Diff: docs/Eventlogging.md ---
    @@ -94,3 +95,29 @@ public interface IEventLogger {
         void close();
     }
     ```
    +
    +The default implementation for this is a FileBasedEventLogger which logs 
the events to an events.log file ( 
`logs/workers-artifacts/<topology-id>/<worker-port>/events.log`).
    +Alternate implementations of the `IEventLogger` interface can be added to 
extend the event logging functionality (say build a search index or log the 
events in a database etc)
    +
    +To register event logger to your topology, add to your topology's 
configuration like:
    +
    +```java
    
+conf.registerEventLogger(org.apache.storm.metric.FileBasedEventLogger.class);
    --- End diff --
    
    nit: remove the metric package name reference


---

Reply via email to