> On 2012-05-02 00:20:27, Arvind Prabhakar wrote: > > +1 Changes look good. A couple of nits noted below.
Resolve both nits and attached to JIRA. Thank you for your review! - Brock ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4838/#review7457 ----------------------------------------------------------- On 2012-04-22 23:17:45, Brock Noland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4838/ > ----------------------------------------------------------- > > (Updated 2012-04-22 23:17:45) > > > Review request for Flume. > > > Summary > ------- > > Message is logged a large volume of times and is incorrect. > > > This addresses bug FLUME-1132. > https://issues.apache.org/jira/browse/FLUME-1132 > > > Diffs > ----- > > > flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSEventSink.java > 1b61cad > > Diff: https://reviews.apache.org/r/4838/diff > > > Testing > ------- > > Logging only change > > > Thanks, > > Brock > >
