----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7700/#review13156 -----------------------------------------------------------
flume-ng-doc/sphinx/FlumeUserGuide.rst <https://reviews.apache.org/r/7700/#comment28312> Generally in flume when we are going to have multiple sub property items we give them a name and then have sub-properties under that name. Thoughts on doing that hear as opposed to using the class name? - Brock Noland On Oct. 22, 2012, 6:26 p.m., Cameron Gandevia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7700/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2012, 6:26 p.m.) > > > Review request for Flume. > > > Description > ------- > > A RegexExtractor interceptor that will allow users to extract regex matches > and append them as header fields of the event. > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorType.java > c478337 > > flume-ng-core/src/main/java/org/apache/flume/interceptor/RegexExtractorInterceptor.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/interceptor/RegexExtractorInterceptorMillisSerializer.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/interceptor/RegexExtractorInterceptorPassThroughSerializer.java > PRE-CREATION > > flume-ng-core/src/main/java/org/apache/flume/interceptor/RegexExtractorInterceptorSerializer.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/interceptor/TestRegexExtractorInterceptor.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/interceptor/TestRegexExtractorInterceptorMillisSerializer.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/interceptor/TestRegexExtractorInterceptorPassThroughSerializer.java > PRE-CREATION > flume-ng-doc/sphinx/FlumeUserGuide.rst bd244d4 > > Diff: https://reviews.apache.org/r/7700/diff/ > > > Testing > ------- > > > Thanks, > > Cameron Gandevia > >
