Skip to content

Commit

Permalink
Merge branch 'conffile'
Browse files Browse the repository at this point in the history
  • Loading branch information
Nathan Marz committed Oct 18, 2012
2 parents 97091a9 + 6cf1734 commit a62807c
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 2 deletions.
9 changes: 8 additions & 1 deletion bin/storm
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ else:
CONF_DIR = os.path.expanduser("~/.storm")
STORM_DIR = "/".join(os.path.realpath( __file__ ).split("/")[:-2])
CONFIG_OPTS = []
CONFFILE = ""

def get_config_opts():
global CONFIG_OPTS
Expand Down Expand Up @@ -52,8 +53,9 @@ def get_classpath(extrajars):
return normclasspath(":".join(ret))

def confvalue(name, extrapaths):
global CONFFILE
command = [
"java", "-client", get_config_opts(), "-cp", get_classpath(extrapaths), "backtype.storm.command.config_value", name
"java", "-client", get_config_opts(), "-Dstorm.conf.file=" + CONFFILE, "-cp", get_classpath(extrapaths), "backtype.storm.command.config_value", name
]
p = sub.Popen(command, stdout=sub.PIPE)
output, errors = p.communicate()
Expand Down Expand Up @@ -104,10 +106,12 @@ def parse_args(string):
return [re.compile(r'\\(.)').sub('\\1', x) for x in args]

def exec_storm_class(klass, jvmtype="-server", jvmopts=[], extrajars=[], args=[], fork=False):
global CONFFILE
all_args = [
"java", jvmtype, get_config_opts(),
"-Dstorm.home=" + STORM_DIR,
"-Djava.library.path=" + confvalue("java.library.path", extrajars),
"-Dstorm.conf.file=" + CONFFILE,
"-cp", get_classpath(extrajars),
] + jvmopts + [klass] + list(args)
print "Running: " + " ".join(all_args)
Expand Down Expand Up @@ -382,6 +386,9 @@ def parse_config_opts(args):
token = curr.pop()
if token == "-c":
config_list.append(curr.pop())
elif token == "--config":
global CONFFILE
CONFFILE = curr.pop()
else:
args_list.append(token)

Expand Down
8 changes: 7 additions & 1 deletion src/jvm/backtype/storm/utils/Utils.java
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,13 @@ public static Map readCommandLineOpts() {

public static Map readStormConfig() {
Map ret = readDefaultConfig();
Map storm = findAndReadConfigFile("storm.yaml", false);
String confFile = System.getProperty("storm.conf.file");
Map storm;
if (confFile==null || confFile.equals("")) {
storm = findAndReadConfigFile("storm.yaml", false);
} else {
storm = findAndReadConfigFile(confFile, true);
}
ret.putAll(storm);
ret.putAll(readCommandLineOpts());
return ret;
Expand Down

0 comments on commit a62807c

Please sign in to comment.