----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5182/#review8022 -----------------------------------------------------------
Ship it! +1 Thanks for the patch Hari. Changes look good. - Arvind On 2012-05-21 23:19:37, Hari Shreedharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5182/ > ----------------------------------------------------------- > > (Updated 2012-05-21 23:19:37) > > > Review request for Flume and Arvind Prabhakar. > > > Summary > ------- > > Added Load balancing channel selector to config type enum > > > This addresses bug FLUME-1220. > https://issues.apache.org/jira/browse/FLUME-1220 > > > Diffs > ----- > > > flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkProcessorConfiguration.java > 341e327 > > Diff: https://reviews.apache.org/r/5182/diff > > > Testing > ------- > > Built, ran unit tests, ran agent with the channel selector. > > > Thanks, > > Hari > >
