Review Board 1.7.22


ExecSource configuration

Review Request #4508 - Created March 27, 2012 and updated

Hari Shreedharan
trunk
FLUME-1058
Reviewers
Flume
flume-git
ExecSource configuration

 
Posted (April 24, 2012, 2:54 a.m.)
Functionally fine, just a couple of niggles
This should probably be a constant
I'm pretty sure that now we have a dedicated  class for the configuration we don't need a subclass to hold the constants?
For consistency I think this should go into the configuration and throw a ConfigurationException