[ 
https://issues.apache.org/jira/browse/STORM-248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266271#comment-14266271
 ] 

ASF GitHub Bot commented on STORM-248:
--------------------------------------

GitHub user fmazoyer opened a pull request:

    https://github.com/apache/storm/pull/373

    STORM-248 STORM-322 are fixed and upmerged

    Details are in JIRA https://issues.apache.org/jira/browse/STORM-248.
    As a collateral benefit, this also addresses JIRA 
https://issues.apache.org/jira/browse/STORM-322.
    Could you please review?

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/fmazoyer/storm STORM-248

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/373.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 #373
    
----
commit b2f8eb651d3e8dbd7939aae126d803da6b948663
Author: fmazoyer <frantz.mazo...@altran.com>
Date:   2014-12-19T16:12:57Z

    STORM-248 STORM-322 proposal for fix

commit 49688fd5856e81d947255aca9cca4252d8243945
Author: fmazoyer <frantz.mazo...@altran.com>
Date:   2014-12-19T16:12:57Z

    STORM-248 STORM-322 proposal for fix

commit 14a828cae8d883a8a0d1022d75156079a37a88a8
Author: fmazoyer <frantz.mazo...@altran.com>
Date:   2015-01-06T12:23:37Z

    Merge branch 'STORM-248' of https://github.com/fmazoyer/storm.git into 
STORM-248

commit fba310af8cc2796af9314bd26217fe08552ad49c
Author: fmazoyer <frantz.mazo...@altran.com>
Date:   2015-01-06T15:27:19Z

    STORM-248 use os.path.join instead or mere path concatenation

----


> cluster.xml location is hardcoded for workers.
> ----------------------------------------------
>
>                 Key: STORM-248
>                 URL: https://issues.apache.org/jira/browse/STORM-248
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Edward Goslin
>            Assignee: Frantz Mazoyer
>            Priority: Trivial
>
> when the supervisor spawns a worker process, it assumes the cluster.xml is in
> -Dlogback.configurationFile=" storm-home "/logback/cluster.xml"
> It should take the VM arguement for the supervisor and pass it down to the 
> worker.
> System.get("logback.configurationFile", storm-home + "/logback/cluster.xml)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to