----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50485/#review143776 -----------------------------------------------------------
Hi Deng Kai, thank you for the patch. Could you please also write a test for this? - Mike Percy On July 27, 2016, 12:46 a.m., deng kai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50485/ > ----------------------------------------------------------- > > (Updated July 27, 2016, 12:46 a.m.) > > > Review request for Flume. > > > Repository: flume-git > > > Description > ------- > > The two flush thread may lead channel into wrong status when channel is > full?With file channel, it will generate massive small log file and > log.meta.tmp if set restart = true. > > > Diffs > ----- > > flume-ng-core/src/main/java/org/apache/flume/source/ExecSource.java 52ea808 > > Diff: https://reviews.apache.org/r/50485/diff/ > > > Testing > ------- > > > Thanks, > > deng kai > >
