----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6737/#review10639 -----------------------------------------------------------
Ship it! Looks good. Filed a jira to move the version numbers out of the pom. This is present in other parts of Flume too, so will do that it one jira. - Hari Shreedharan On Aug. 22, 2012, 7:30 p.m., Brock Noland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6737/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2012, 7:30 p.m.) > > > Review request for Flume and Hari Shreedharan. > > > Description > ------- > > Instead of sleeping until a checkpoint occurs, we can force a checkpoint. > > > This addresses bug FLUME-1505. > https://issues.apache.org/jira/browse/FLUME-1505 > > > Diffs > ----- > > flume-ng-channels/flume-file-channel/pom.xml cd882e5 > > flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/Log.java > 9b13423 > > flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFileChannel.java > 3dededf > > Diff: https://reviews.apache.org/r/6737/diff/ > > > Testing > ------- > > Unit tests pass. > > > Thanks, > > Brock Noland > >
