-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4010/
-----------------------------------------------------------

(Updated 2012-02-23 09:07:56.494266)


Review request for Flume.


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-968.
    https://issues.apache.org/jira/browse/FLUME-968


Diffs
-----

  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

Reply via email to