-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4010/
-----------------------------------------------------------
(Updated 2012-02-23 09:07:12.776308)
Review request for Flume.
Changes
-------
All changes
Summary
-------
This is a first cut for the library that does the syntactical validation of
config files. I would like feedback on naming conventions for the errors and
better ways to pass data around. Basically this patch is meant to demonstrate
the fundamental idea.
This addresses bug FLUME-969.
https://issues.apache.org/jira/browse/FLUME-969
Diffs (updated)
-----
flume-ng-configuration/pom.xml PRE-CREATION
flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfiguration.java
PRE-CREATION
flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfigurationError.java
PRE-CREATION
flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfigurationErrorType.java
PRE-CREATION
flume-ng-node/pom.xml b9b062e
flume-ng-node/src/main/java/org/apache/flume/conf/properties/FlumeConfiguration.java
d66f6d1
flume-ng-node/src/main/java/org/apache/flume/conf/properties/PropertiesFileConfigurationProvider.java
97f72e1
pom.xml d785762
Diff: https://reviews.apache.org/r/4010/diff
Testing
-------
I will add tests once I get initial feedback.
Thanks,
Hari
