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

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

GitHub user arunmahadevan opened a pull request:

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

    [STORM-1024] Pass -Dstorm.log.dir arg to LogWriter

    The log4j variable `${sys:storm.log.dir}` is used in `worker.xml` to set 
the fileName however the system property was not being set while launching 
worker (via LogWriter) causing a directory by the name ${sys:storm.log.dir} to 
be created under `STORM_HOME` dir.
    
    This fix is to pass the system property while launching LogWriter

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

    $ git pull https://github.com/arunmahadevan/storm master

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

    https://github.com/apache/storm/pull/714.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 #714
    
----
commit 6c2b754a7a0c69852850c4dd39af31ba98ef847d
Author: Arun Mahadevan <ai...@hortonworks.com>
Date:   2015-09-02T17:31:13Z

    Pass -Dstorm.log.dir arg to LogWriter

----


> log4j changes leaving ${sys:storm.log.dir} under STORM_HOME dir
> ---------------------------------------------------------------
>
>                 Key: STORM-1024
>                 URL: https://issues.apache.org/jira/browse/STORM-1024
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Sriharsha Chintalapani
>            Assignee: Arun Mahadevan
>




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

Reply via email to