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

Reply via email to