----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14439/#review26853 -----------------------------------------------------------
Ship it! +1 looks good. Thanks for the quick follow-up Jeff! - Mike Percy On Oct. 10, 2013, 1:35 a.m., Jeff jlord wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14439/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2013, 1:35 a.m.) > > > Review request for Flume. > > > Repository: flume-git > > > Description > ------- > > Attaching a patch which introduces a boolean keepFields which defaults to > false. When set to true this will preserve the timestamp and hostname in the > body of the event. Additionally I have added a test for SyslogTcpSource > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/source/SyslogSourceConfigurationConstants.java > 5a73c88 > flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java > db9e0fd > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java > c2a29a1 > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogTcpSource.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java > 2d7a429 > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUtils.java > 7208464 > flume-ng-doc/sphinx/FlumeDeveloperGuide.rst 2be9c68 > flume-ng-doc/sphinx/FlumeUserGuide.rst dac3ce7 > > Diff: https://reviews.apache.org/r/14439/diff/ > > > Testing > ------- > > > Thanks, > > Jeff jlord > >
