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

(Updated 2012-04-19 01:23:41.296962)


Review request for Flume.


Changes
-------

Incorporated Juhani's feedback. For some reason, git seems to have reordered 
the files in the diff, sorry about that.


Summary
-------

Main config component.


This addresses bug FLUME-1052.
    https://issues.apache.org/jira/browse/FLUME-1052


Diffs (updated)
-----

  flume-ng-core/pom.xml b9f1e12 
  flume-ng-core/src/main/java/org/apache/flume/ChannelSelector.java fba2dcb 
  flume-ng-core/src/main/java/org/apache/flume/Context.java 5294e31 
  flume-ng-core/src/main/java/org/apache/flume/SinkProcessor.java d863ed0 
  flume-ng-core/src/main/java/org/apache/flume/SinkProcessorType.java be1891b 
  
flume-ng-core/src/main/java/org/apache/flume/channel/ChannelSelectorFactory.java
 800f471 
  flume-ng-core/src/main/java/org/apache/flume/channel/ChannelSelectorType.java 
511fc65 
  flume-ng-core/src/main/java/org/apache/flume/channel/ChannelType.java d8419e8 
  
flume-ng-core/src/main/java/org/apache/flume/channel/DefaultChannelFactory.java 
963a6a3 
  flume-ng-core/src/main/java/org/apache/flume/conf/Configurables.java 84492e5 
  flume-ng-core/src/main/java/org/apache/flume/sink/DefaultSinkFactory.java 
b89dfa0 
  flume-ng-core/src/main/java/org/apache/flume/sink/DefaultSinkProcessor.java 
257bab3 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkGroup.java 0dffd69 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkProcessorFactory.java 
6160a17 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkType.java 6b08c09 
  flume-ng-core/src/main/java/org/apache/flume/source/DefaultSourceFactory.java 
a610e6f 
  flume-ng-core/src/main/java/org/apache/flume/source/SourceType.java cd8991e 
  flume-ng-dist/pom.xml 642e681 
  flume-ng-dist/src/main/assembly/dist.xml 917ac02 
  flume-ng-dist/src/main/assembly/src.xml 3b24b39 
  
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
 1f0e8c6 
  pom.xml ed8092d 

Diff: https://reviews.apache.org/r/4502/diff


Testing
-------

Functional testing done.


Thanks,

Hari

Reply via email to