GitHub user zjffdu opened a pull request:

    https://github.com/apache/zeppelin/pull/1333

    [ZEPPELIN-1334] Environment variable defined in interpreter setting doesn't 
take effect

    ### What is this PR for?
    I define SPAKR_HOME in interpreter setting, but it doesn't take effect. 
This PR is for bring back the environment variable defined in interpreter 
setting.  The root cause is that we reset the env after creating 
RemoteInterpreter.
    ```
             new RemoteInterpreter(property, noteId, className, 
conf.getInterpreterRemoteRunnerPath(),
                 interpreterPath, localRepoPath, connectTimeout, maxPoolSize,
                 remoteInterpreterProcessListener, appEventListener);
        remoteInterpreter.setEnv(env);
    ```
    
    
    ### What type of PR is it?
    [Bug Fix]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-1334
    
    ### How should this be tested?
    Tested manually.  Create 2 spark interpreter setting, one for spark1 
another is for spark2. And define SPARK_HOME for each interpreter. Then I can 
run both spark1 and spark2 in one zeppelin instance. 
    
    ### Screenshots (if appropriate)
    
![image](https://cloud.githubusercontent.com/assets/164491/17696073/b64b1014-63de-11e6-88ab-d26b1c2fa75c.png)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    


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

    $ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1334

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

    https://github.com/apache/zeppelin/pull/1333.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 #1333
    
----
commit 3e7c1fabf364c8b6cfcb2b28fcfac0618cfb36aa
Author: Jeff Zhang <[email protected]>
Date:   2016-08-16T10:16:58Z

    [ZEPPELIN-1334] Environment variable defined in interpreter setting doesn't 
take effect

----


---
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.
---

Reply via email to