[ 
https://issues.apache.org/jira/browse/FLUME-1170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13267299#comment-13267299
 ] 

Hari Shreedharan commented on FLUME-1170:
-----------------------------------------

Thanks for the comments Arvind. 

I thought about this and I decided it might be a good idea to add it on both 
sides, with configurability and functions that will be called either on put or 
take. 
Basically:
* provide a way to configure if the callbacks should be made on put, get or 
both. 
* provide an interface with the following callback functions:
   * onPut(Map<String, String> eventHeaders), called on put, if the config 
specifies put or both.
   * onTake(Map<String, String> eventHeaders), called on take, based on config.
   * onCommit()
   * onRollback()

I am adding support in MemoryChannel and RecoverableMemory channel and a unit 
test for MemoryChannel with txnlistener support.
                
> Create a framework for transaction listeners
> --------------------------------------------
>
>                 Key: FLUME-1170
>                 URL: https://issues.apache.org/jira/browse/FLUME-1170
>             Project: Flume
>          Issue Type: New Feature
>          Components: Channel, Sinks+Sources
>    Affects Versions: v1.2.0
>            Reporter: Hari Shreedharan
>            Assignee: Hari Shreedharan
>             Fix For: v1.2.0
>
>
> Initially support on the sink side. It cannot be added to the channel as it 
> is possible to support transaction listeners on both source and sink, and 
> each side could be a different listeners. As of now, support on sink side 
> transactions. It should be fairly easy to add this to source side, if we need 
> to.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to