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

(Updated 2012-03-30 21:38:31.640395)


Review request for Flume.


Changes
-------

Updated patch so that we can now specify -Dflume.root.logger=DEBUG,console when 
invoking Flume on the command line, which overrides the log4j.rootLogger. I 
think that's a reasonable thing to ask of developers. At the same time, "out of 
the box", Flume has logging that is sane for a production environment.


Summary
-------

Simple patch to default to a reasonably-configured rolling file appender.


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


Diffs (updated)
-----

  bin/flume-ng a31f171 
  conf/log4j.properties 3dcffd4 

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


Testing
-------

Did some manual testing, looks fine to me.


Thanks,

Mike

Reply via email to