-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24398/
-----------------------------------------------------------
Review request for Flume.
Bugs: FLUME-1491
https://issues.apache.org/jira/browse/FLUME-1491
Repository: flume-git
Description
-------
Add static and polling ZooKeeper configuration providers
Add test cases
Update command-line usage printout
Diffs
-----
bin/flume-ng e09e26b
flume-ng-node/pom.xml dce2527
flume-ng-node/src/main/java/org/apache/flume/node/AbstractConfigurationProvider.java
e63c601
flume-ng-node/src/main/java/org/apache/flume/node/AbstractZooKeeperConfigurationProvider.java
PRE-CREATION
flume-ng-node/src/main/java/org/apache/flume/node/Application.java 5250139
flume-ng-node/src/main/java/org/apache/flume/node/PollingZooKeeperConfigurationProvider.java
PRE-CREATION
flume-ng-node/src/main/java/org/apache/flume/node/PropertiesFileConfigurationProvider.java
d7438d9
flume-ng-node/src/main/java/org/apache/flume/node/StaticZooKeeperConfigurationProvider.java
PRE-CREATION
flume-ng-node/src/test/java/org/apache/flume/node/TestAbstractZooKeeperConfigurationProvider.java
PRE-CREATION
flume-ng-node/src/test/java/org/apache/flume/node/TestPollingZooKeeperConfigurationProvider.java
PRE-CREATION
flume-ng-node/src/test/java/org/apache/flume/node/TestStaticZooKeeperConfigurationProvider.java
PRE-CREATION
pom.xml 4bdfcac
Diff: https://reviews.apache.org/r/24398/diff/
Testing
-------
Verified polling ZK config provider in live setup by updating configuration
dynamically and watching the changes take effect
Verified static ZK config provider in live setup by starting with
--no-reload-conf
Verified test cases for polling and static pass
Thanks,
Zachary Heilbron
