----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50562/#review144100 -----------------------------------------------------------
flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java (line 82) <https://reviews.apache.org/r/50562/#comment210111> please remove test logging statement flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java (lines 89 - 103) <https://reviews.apache.org/r/50562/#comment210112> looks good flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java (lines 131 - 146) <https://reviews.apache.org/r/50562/#comment210113> looks good flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java (lines 181 - 219) <https://reviews.apache.org/r/50562/#comment210114> looks good generally looks good, I vote for shipping it after leftover log statement was removed - Attila Simon On July 28, 2016, 11:46 a.m., Denes Arvay wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50562/ > ----------------------------------------------------------- > > (Updated July 28, 2016, 11:46 a.m.) > > > Review request for Flume, Balázs Donát Bessenyei, Mike Percy, and Attila > Simon. > > > Bugs: FLUME-2797 > https://issues.apache.org/jira/browse/FLUME-2797 > > > Repository: flume-git > > > Description > ------- > > Patch for FLUME-2797 - SyslogTcpSource uses Deprecated Class + Deprecate > SyslogTCPSource > > > Diffs > ----- > > flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java > 185c00c > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java > 175bebb > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogTcpSource.java > 10ef8d8 > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java > e5b7a06 > > Diff: https://reviews.apache.org/r/50562/diff/ > > > Testing > ------- > > - checkstyle passes > - `flume-ng-core` tests pass > - 2 new tests added > > > Thanks, > > Denes Arvay > >
