----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50232/#review145074 -----------------------------------------------------------
Fix it, then Ship it! One comment otherwise LGTM flume-ng-core/src/main/java/org/apache/flume/source/SpoolDirectorySource.java (line 285) <https://reviews.apache.org/r/50232/#comment211257> please choose a more descriptive name for this, e.g. `waitAndGetNewBackoffInterval`, or similar. - Denes Arvay On Aug. 3, 2016, 3:54 p.m., Balázs Donát Bessenyei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50232/ > ----------------------------------------------------------- > > (Updated Aug. 3, 2016, 3:54 p.m.) > > > Review request for Flume, Denes Arvay and Attila Simon. > > > Repository: flume-git > > > Description > ------- > > Spooldir assumes that any ChannelException means that the channel is full and > it does not log the exception message. > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/source/SpoolDirectorySource.java > d88cc1d > > flume-ng-core/src/test/java/org/apache/flume/source/TestSpoolDirectorySource.java > 82c5351 > > Diff: https://reviews.apache.org/r/50232/diff/ > > > Testing > ------- > > [INFO] Flume NG Core ...................................... SUCCESS [08:04 > min] > > > Thanks, > > Balázs Donát Bessenyei > >
