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

(Updated April 15, 2013, 2:57 a.m.)


Review request for Flume.


Changes
-------

Hi Jarcec, thanks for the whitespace catch. I fixed that.

Regarding the command-line option, you specify it as a colon-separated list of 
directories.

I just renamed it form --plugins-dirs to --plugins-path to hopefully make that 
less confusing.

Example usage:

./bin/flume-ng agent -c ~/work/flume/conf -f 
~/work/flume/conf/flume.pluginsd-test.properties -n agent 
-Dflume.root.logger=DEBUG,console --plugins-path 
"$HOME/work/flume/plugins.d.1:$HOME/work/flume/plugins.d.2"


Description
-------

Added capability to specify one or more plugins directories on the command line.


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


Diffs (updated)
-----

  bin/flume-ng ee86c95 

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


Testing
-------

Manually tested with config file and 2 plugins.d directories


Thanks,

Mike Percy

Reply via email to