----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4173/#review6200 -----------------------------------------------------------
Ship it! +1 (ignoring the whitespace issue that I will plan to run a sweep for soon). - Arvind On 2012-03-05 03:51:43, Juhani Connolly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4173/ > ----------------------------------------------------------- > > (Updated 2012-03-05 03:51:43) > > > Review request for Flume. > > > Summary > ------- > > Fixed documentation to make it clear that priorities must be unique. > > Also added a logging warning when a non-unique priority is encountered, > ignoring the sink that would overwrite. > > > This addresses bug FLUME-1002. > https://issues.apache.org/jira/browse/FLUME-1002 > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/sink/FailoverSinkProcessor.java > 9f5b856 > > Diff: https://reviews.apache.org/r/4173/diff > > > Testing > ------- > > Run full test suite, all clear > > > Thanks, > > Juhani > >
