----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8413/#review14176 -----------------------------------------------------------
Nice work Hari! I'll run the tests and commit if they pass! - Brock Noland On Dec. 7, 2012, 7:12 p.m., Hari Shreedharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8413/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2012, 7:12 p.m.) > > > Review request for Flume. > > > Description > ------- > > Replicating channel selector should support optional channels > > > This addresses bug FLUME-1769. > https://issues.apache.org/jira/browse/FLUME-1769 > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/channel/AbstractChannelSelector.java > 939257d > > flume-ng-core/src/main/java/org/apache/flume/channel/MultiplexingChannelSelector.java > 866d9dc > > flume-ng-core/src/main/java/org/apache/flume/channel/ReplicatingChannelSelector.java > 8f22746 > > flume-ng-core/src/test/java/org/apache/flume/channel/TestReplicatingChannelSelector.java > e671364 > flume-ng-doc/sphinx/FlumeUserGuide.rst f8528bb > > Diff: https://reviews.apache.org/r/8413/diff/ > > > Testing > ------- > > Added unit tests to handle this case > > > Thanks, > > Hari Shreedharan > >
