GitHub user zjffdu opened a pull request:

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

    ZEPPELIN-3531. Don't look for py4j in NewSparkInterpreter

    ### What is this PR for?
    Just remove setupConfForPySpark in NewSparkInterpreter as it is not 
necessary and will cause NPE when the node launch spark interpreter doesn't 
have spark installed in yarn cluster mode.
    
    
    ### What type of PR is it?
    [Bug Fix]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-3531
    
    ### How should this be tested?
    * CI pass & Manually tested in a 3 node cluster
    
    
    ### Screenshots (if appropriate)
    
    ### 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-3531

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

    https://github.com/apache/zeppelin/pull/3008.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 #3008
    
----
commit d3fa2d2e2808090032678b0baddb2281ce81f582
Author: Jeff Zhang <zjffdu@...>
Date:   2018-06-05T13:38:17Z

    ZEPPELIN-3531. Don't look for py4j in NewSparkInterpreter

----


---

Reply via email to