----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52216/#review150299 -----------------------------------------------------------
Fix it, then Ship it! flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFileChannel.java (line 650) <https://reviews.apache.org/r/52216/#comment218204> style nit: can you please join the first two strings? flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFileChannel.java (line 656) <https://reviews.apache.org/r/52216/#comment218205> same here - Lior Zeno On Sept. 24, 2016, 12:23 p.m., Umesh Chaudhary wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52216/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2016, 12:23 p.m.) > > > Review request for Flume. > > > Repository: flume-git > > > Description > ------- > > Added two new booleans to identify the reason for closed channel: 1) > isClosedOnStart 2) isClosedNormally > And checked them to print the appropriate reason for closed channel > > > Diffs > ----- > > > flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/FileChannel.java > 9d82e43 > > flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFileChannel.java > bfc2d0d > > Diff: https://reviews.apache.org/r/52216/diff/ > > > Testing > ------- > > Yes > > > Thanks, > > Umesh Chaudhary > >
