-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8421/
-----------------------------------------------------------
(Updated Dec. 12, 2012, 7:30 p.m.)
Review request for Flume.
Changes
-------
Latest patch which addresses Hari's concerns.
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 (updated)
-----
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
