-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14569/
-----------------------------------------------------------
(Updated Oct. 24, 2013, 7:47 a.m.)
Review request for Flume.
Changes
-------
Hi Mike: No worries! I have rebased the code and regenerated the diff file.
Pls. review and let me know if you have any feedback/suggestions.
Thanks
Venkatesh
Bugs: Flume-2120
https://issues.apache.org/jira/browse/Flume-2120
Repository: flume-git
Description
-------
Pls. find the patch attached for the FLUME-2120.
I have made changes to SyslogUDPSource and SyslogTCPSource. Tested it locally
and also have added a JUnit for SyslogUDPSource.
I see the MultiportSyslogTCPSource is pretty much a better version/likely
replacement for SyslogTCPSource. But still went ahead and added the metrics
counters to SyslogTCPSource, as the change was straight forward and its very
much a part of the code base.
Diffs (updated)
-----
flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java
7a12d27
flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java
96a9e85
flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogTcpSource.java
a6a1d5b
flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java
eae26ed
Diff: https://reviews.apache.org/r/14569/diff/
Testing
-------
JUnit added for SyslogUDPSource and SyslogTcpSource
Thanks,
Venkatesh Sivasubramanian
