FLUME-281: Allow System property-based specification of FLUME_HOME/FLUME_CONF_DIR

Review Request #1048 - Created Oct. 19, 2010 and submitted

Aaron Kimball
old-flume
FLUME-281
flume
Allows users to specify -Dflume.home and -Dflume.conf.dir instead of $FLUME_HOME/$FLUME_CONF_DIR.
Adds new unit test. All existing unit tests pass.
Also embedded Flume in my application in an application server without $FLUME_HOME set in the environment. Flume works with this patch + system-property setting, and doesn't work without it.
  1. Minor nits. Looks good. I'll try and commit later tonight.
  2. Super minor nit: The majority of the code base uses variable on the left, constant on the right.
  3. 
      
Review request changed

Change Summary:

updated title

Summary:

-Allow System property-based specification of FLUME_HOME/FLUME_CONF_DIR
+FLUME-281: Allow System property-based specification of FLUME_HOME/FLUME_CONF_DIR
  1. Sorry - I reviewed this but never gave it a ship it. It's already in master.
  2. 
      
Loading...