[FLUME-537] Error message for "exec load" could be more clear

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Won't Fix
• Affects Version/s: 0.9.3
• Fix Version/s: 0.9.5
• Component/s: Master

Environment

Ubuntu 10.04 LTS on EC2

• Assignee: Unassigned
• Reporter: Disabled imported user
• Created: 23/Feb/11 23:27
• Updated: 05/Nov/14 09:39
• Resolved: 05/Nov/14 09:39

Description

We upgraded from 0.9.1 to 0.9.3 and our "exec load" failed:

 
ubuntu@domU-12-31-39-04-54-CE:~$ flume shell -c localhost -e "exec load"
2011-02-23 23:25:17,006 [main] INFO conf.FlumeConfiguration: Loading configurations from /etc/flume/conf
Using default admin port: 35873
Using default report port: 45678
Connecting to Flume master localhost:35873:45678...
2011-02-23 23:25:17,168 [main] INFO util.AdminRPCThrift: Connected to master at localhost:35873
[id: 16] Execing command : load
Command failed
null

It looks like LoadConfigCommand changed to require a file versus assuming current.flume as the default.

This line looks to be the culprit - perhaps missing an error message if the assertion fails?

 
Preconditions.checkArgument(argv.length == 0);

Sharethrough loves Flume!

Rob