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

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

GitHub user sureshms opened a pull request:

    https://github.com/apache/incubator-storm/pull/85

    STORM-295. Add storm configuration to define JAVA_HOME

    Here is a patch that adds support for JAVA_HOME. The patch does the 
following:
    # New file conf/storm-env.sh is added to setup environment. This is where 
JAVA_HOME is set.
    # I moved storm python script to storm.py. A new shell script storm sets up 
the environment variables and launches storm.py. 
    # Storm.py now uses java path derived from JAVA_HOME or uses java from PATH 
if JAVA_HOME is not set.

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

    $ git pull https://github.com/sureshms/incubator-storm STORM-295

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

    https://github.com/apache/incubator-storm/pull/85.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 #85
    
----
commit d7b74d030c4e4212c9241b6f387304deb128848f
Author: P. Taylor Goetz <[email protected]>
Date:   2014-02-19T22:26:23Z

    STORM-295. Add storm configuration to define JAVA_HOME

----


> Add storm configuration to define JAVA_HOME
> -------------------------------------------
>
>                 Key: STORM-295
>                 URL: https://issues.apache.org/jira/browse/STORM-295
>             Project: Apache Storm (Incubating)
>          Issue Type: Bug
>            Reporter: Suresh Srinivas
>
> Currently storm uses either /usr/bin/java (most likely) or java pointed to by 
> PATH.
> Storm requires a way to define JAVA_HOME in configuration. This will simplify 
> pointing to a version of java that a cluster admin or test frameworks intend 
> to use for storm, especially in an environments where multiple versions of 
> java is on the cluster nodes.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to