GitHub user clockfly opened a pull request:
https://github.com/apache/incubator-storm/pull/120
STORM-188: Allow user to specifiy full configuration path when running
storm command
We can use system config "storm.conf.file" to specify a custom config file.
Before this patch, we will only try to lookup this file on jvm classpath.
This patch will give us the ability to specify full/relative filesystem
path as configuration file path, thus will give us more flexibility, while we
still are able to look it up on jvm classpath.
Now we are able to submit storm topology like this:
```bash
storm jar job.jar --config /tmp/dynamic-configuration-path.yaml
```
We don't need to add /tmp into our classpath(also inappropriate).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clockfly/incubator-storm
storm-188-storm-config-support-localfs-path
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/120.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #120
----
commit 0512fb461504d31f3011417e9ac66bc7b4a8f8e1
Author: Sean Zhong <[email protected]>
Date: 2014-05-21T02:03:33Z
STORM-188: when looking up storm.yaml, should also try to search file
system path if we cannot find it on classpath
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
