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

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


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

Ship it!


+1

Please rebase the patch to the latest trunk sources.

- Arvind


On 2012-05-11 08:17:26, Mike Percy wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/5091/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-05-11 08:17:26)
bq.  
bq.  
bq.  Review request for Flume.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Implementation of Source-side Decorators (called Interceptors here) for 
Flume 1.x.
bq.  
bq.  
bq.  This addresses bug FLUME-1157.
bq.      https://issues.apache.org/jira/browse/FLUME-1157
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
flume-ng-core/src/test/java/org/apache/flume/interceptor/TestCensoringInterceptor.java
 PRE-CREATION 
bq.    
flume-ng-core/src/main/java/org/apache/flume/source/PollableSourceRunner.java 
6ffdb5f 
bq.    
flume-ng-core/src/test/java/org/apache/flume/interceptor/CensoringInterceptor.java
 PRE-CREATION 
bq.    
flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorChain.java 
PRE-CREATION 
bq.    
flume-ng-core/src/main/java/org/apache/flume/source/EventDrivenSourceRunner.java
 9dd15d4 
bq.    
flume-ng-core/src/main/java/org/apache/flume/interceptor/Interceptor.java 
PRE-CREATION 
bq.    
flume-ng-core/src/main/java/org/apache/flume/channel/ChannelProcessor.java 
eb6460e 
bq.    
flume-ng-core/src/test/java/org/apache/flume/source/TestPollableSourceRunner.java
 c27f82c 
bq.    
flume-ng-node/src/main/java/org/apache/flume/conf/properties/PropertiesFileConfigurationProvider.java
 878feda 
bq.  
bq.  Diff: https://reviews.apache.org/r/5091/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added a unit test with an example Interceptor. Existing unit tests pass.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Mike
bq.  
bq.


                
> 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