Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Flume, mail # dev - Re: Review Request 14439: FLUME-1666 Syslog source strips timestamp and hostname from log message body


+
Jeff jlord 2013-10-10, 01:14
+
Jeff jlord 2013-10-10, 01:35
Copy link to this message
-
Re: Review Request 14439: FLUME-1666 Syslog source strips timestamp and hostname from log message body
Mike Percy 2013-10-10, 01:36

-----------------------------------------------------------
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
>
>