----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8845/#review15330 -----------------------------------------------------------
Ship it! - Brock Noland On Jan. 14, 2013, 7:35 p.m., Hari Shreedharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8845/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2013, 7:35 p.m.) > > > Review request for Flume. > > > Description > ------- > > Added layout support to log4jappender. > > > This addresses bug FLUME-1818. > https://issues.apache.org/jira/browse/FLUME-1818 > > > Diffs > ----- > > > 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 > >
