----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8421/#review14202 -----------------------------------------------------------
Generally looks good. I have a couple of comments: * There are a lot of tabs/trailing spaces. Could you please remove those. * If we remove these components, we need to make sure that after all components are loaded/removed, we have at least 1 channel with a source or sink in the agent. Basically after all load* methods are done, we should iterate through the sinks and sources and verify their channels(in case of sources, at least one of the channels) are still around (if not the source/sink should be removed), else the config is invalid. - Hari Shreedharan On Dec. 7, 2012, 8:37 p.m., Brock Noland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8421/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2012, 8:37 p.m.) > > > Review request for Flume. > > > Description > ------- > > Catches Exception around all the configure() methods of components. If they > throw an exception, they are removed from the configuration (not added). > > > This addresses bug FLUME-1772. > https://issues.apache.org/jira/browse/FLUME-1772 > > > Diffs > ----- > > > flume-ng-node/src/main/java/org/apache/flume/node/AbstractConfigurationProvider.java > daef76b > > flume-ng-node/src/test/java/org/apache/flume/node/TestAbstractConfigurationProvider.java > 25001b1 > > Diff: https://reviews.apache.org/r/8421/diff/ > > > Testing > ------- > > Tests added for source, channel, and sink. > > > Thanks, > > Brock Noland > >
