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

Switch to Threaded View
Flume, mail # dev - Review Request: FLUME-1818. Add layout support to log4jappender.


Copy link to this message
-
Re: Review Request: FLUME-1818. Add layout support to log4jappender.
Hari Shreedharan 2013-01-06, 02:38

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

(Updated Jan. 6, 2013, 2:38 a.m.)
Review request for Flume.
Changes
-------

Updated unit tests to make sure different logging levels are actually used (since log4j levels are multiples of 10000).
Description
-------

Added layout support to log4jappender.
This addresses bug FLUME-1818.
    https://issues.apache.org/jira/browse/FLUME-1818
Diffs (updated)
-----

  flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAppender.java 083f5d1
  flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLog4jAppender.java 68d95fb

Diff: https://reviews.apache.org/r/8845/diff/
Testing
-------

Existing tests pass, added new test to test this feature.
Thanks,

Hari Shreedharan