GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/2457
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
This is in reality backward compatible change (though it is 'technically'
not backward compatible), and I'd like to get it in 1.x version as well.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/HeartSaVioR/storm STORM-2854
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2457.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 #2457
----
commit 588349ccac7bb68effd67e33af01eeb76bd4e447
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
----
---
