GitHub user HeartSaVioR opened a pull request:

    https://github.com/apache/storm/pull/2458

    (1.x) STORM-2854 Expose IEventLogger to make event logging pluggable

    * expose option to register IEventLogger similar to metrics consumer
    * change the interface of IEventLogger slightly
      * allow argument
      * the change is technically not backward compatible but in real we can 
treat it's OK
        * cause we don't provide a chance to implement custom IEventLogger and 
plug to topology
    * document the change


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/HeartSaVioR/storm STORM-2854-1.x

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/2458.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2458
    
----
commit 12f8e1f2a8adbeede8b8a94dc0c4dcbc15e49768
Author: Jungtaek Lim <[email protected]>
Date:   2017-12-14T06:53:05Z

    STORM-2854 Expose IEventLogger to make event logging pluggable
    
    * expose option to register IEventLogger similar to metrics consumer
    * change the interface of IEventLogger slightly
      * allow argument
      * the change is technically not backward compatible but in real we can 
treat it's OK
        * cause we don't provide a chance to implement custom IEventLogger and 
plug to topology
    * document the change

----


---

Reply via email to