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

[email protected] commented on FLUME-1157:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5091/
-----------------------------------------------------------

(Updated 2012-05-15 21:32:12.934460)


Review request for Flume.


Changes
-------

rebased patch


Summary
-------

Implementation of Source-side Decorators (called Interceptors here) for Flume 
1.x.


This addresses bug FLUME-1157.
    https://issues.apache.org/jira/browse/FLUME-1157


Diffs (updated)
-----

  flume-ng-core/src/main/java/org/apache/flume/channel/ChannelProcessor.java 
eb6460e 
  
flume-ng-core/src/main/java/org/apache/flume/source/EventDrivenSourceRunner.java
 9dd15d4 
  flume-ng-core/src/main/java/org/apache/flume/source/PollableSourceRunner.java 
6ffdb5f 
  
flume-ng-core/src/test/java/org/apache/flume/source/TestPollableSourceRunner.java
 c27f82c 
  
flume-ng-node/src/main/java/org/apache/flume/conf/properties/PropertiesFileConfigurationProvider.java
 82c5dd3 

Diff: https://reviews.apache.org/r/5091/diff


Testing
-------

Added a unit test with an example Interceptor. Existing unit tests pass.


Thanks,

Mike


                
> Implement Decorators for Flume 1.x
> ----------------------------------
>
>                 Key: FLUME-1157
>                 URL: https://issues.apache.org/jira/browse/FLUME-1157
>             Project: Flume
>          Issue Type: New Feature
>            Reporter: Arvind Prabhakar
>            Assignee: Mike Percy
>             Fix For: v1.2.0
>
>         Attachments: FLUME-1157-1.patch
>
>
> Some nice to have built in decorators could be:
> * checksum decorator
> * checksum validation decorator 
> * timestamp decorator
> * GUID decorator
> The implementation should support the following:
> * support multiple decorators in predefined order via configuration
> * support custom decorators.

--
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