[GitHub] zeppelin pull request #3008: ZEPPELIN-3531. Don't look for py4j in NewSparkI...

2018-06-07 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[GitHub] zeppelin pull request #3008: ZEPPELIN-3531. Don't look for py4j in NewSparkI...

2018-06-05 Thread zjffdu
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 
Date:   2018-06-05T13:38:17Z

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




---