----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3831/#review5168 -----------------------------------------------------------
Ship it! - Brock On 2012-02-16 21:49:51, Prasad Mujumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3831/ > ----------------------------------------------------------- > > (Updated 2012-02-16 21:49:51) > > > Review request for Flume. > > > Summary > ------- > > syslog source support for Flume 1.x > > > This addresses bug FLUME-892. > https://issues.apache.org/jira/browse/FLUME-892 > > > Diffs > ----- > > flume-ng-core/pom.xml d753fa1 > flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java > PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java > PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/3831/diff > > > Testing > ------- > > regression tests > Added new test for UDP soruce. tested the TCP source manually. Haven't found > a package that has the tcp syslog client. will add a testcase with custom tcp > client to test the source. > > > Thanks, > > Prasad > >
