----------------------------------------------------------- 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: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5091/ > ----------------------------------------------------------- > > (Updated 2012-05-11 08:17:26) > > > Review request for Flume. > > > 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 > ----- > > > flume-ng-core/src/test/java/org/apache/flume/interceptor/TestCensoringInterceptor.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/source/PollableSourceRunner.java > 6ffdb5f > > flume-ng-core/src/test/java/org/apache/flume/interceptor/CensoringInterceptor.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorChain.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/source/EventDrivenSourceRunner.java > 9dd15d4 > flume-ng-core/src/main/java/org/apache/flume/interceptor/Interceptor.java > PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/channel/ChannelProcessor.java > eb6460e > > 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 > 878feda > > Diff: https://reviews.apache.org/r/5091/diff > > > Testing > ------- > > Added a unit test with an example Interceptor. Existing unit tests pass. > > > Thanks, > > Mike > >
