-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4809/
-----------------------------------------------------------

(Updated 2012-04-27 14:58:21.377103)


Review request for Flume, Arvind Prabhakar and Mike Percy.


Changes
-------

Refactored to use regex for syslog header format. Also additional config to 
specify add-hoc format 


Summary
-------

Support for timestamp and hostname parsing.


This addresses bug FLUME-1126.
    https://issues.apache.org/jira/browse/FLUME-1126


Diffs (updated)
-----

  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 
  flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUtils.java 
8b1f7c5 

Diff: https://reviews.apache.org/r/4809/diff


Testing
-------

Updated syslog tests to cover timestamp and hostname handling.
Manually tested using syslog4j


Thanks,

Prasad

Reply via email to