----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4809/#review7053 -----------------------------------------------------------
Prasad, I get the following error. Patch needs a rebase against trunk? $ patch -p0 < FLUME-1126-5.patch patching file flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java patching file flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUtils.java patch: **** malformed patch at line 315: diff --git flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java - Mike On 2012-04-19 23:01:28, Prasad Mujumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4809/ > ----------------------------------------------------------- > > (Updated 2012-04-19 23:01:28) > > > Review request for Flume, Arvind Prabhakar and Mike Percy. > > > Summary > ------- > > Support for timestamp and hostname parsing. > > > This addresses bug FLUME-1126. > https://issues.apache.org/jira/browse/FLUME-1126 > > > Diffs > ----- > > > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java > 3a7c486 > flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUtils.java > 8b1f7c5 > flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java > b0485b1 > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java > 732cce5 > flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java > 653f5eb > > Diff: https://reviews.apache.org/r/4809/diff > > > Testing > ------- > > Updated syslog tests to cover timestamp and hostname handling. > Manually tested using syslog4j > > > Thanks, > > Prasad > >
