[GitHub] zeppelin pull request #2581: Fix/jdbc connection release 0.7

[astroshim]

Github user astroshim closed the pull request at:

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


---

[GitHub] zeppelin pull request #2581: Fix/jdbc connection release 0.7

[astroshim]

GitHub user astroshim opened a pull request:

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

Fix/jdbc connection release 0.7

### What is this PR for?
This PR fixes JDBC connection release problem.

for example whenever i run not executable command like following

![image](https://user-images.githubusercontent.com/3348133/30206892-8248d1ae-94c8-11e7-9eae-a495be075892.png)

new JDBC connection is made like following.
```
$ netstat -an |grep EST |grep 3306 |wc -l
   1
$ netstat -an |grep EST |grep 3306 |wc -l
   2
$ netstat -an |grep EST |grep 3306 |wc -l
   3
```


### What type of PR is it?
Bug Fix


### 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/astroshim/zeppelin 
fix/jdbcConnectionRelease-0.7

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

https://github.com/apache/zeppelin/pull/2581.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 #2581


commit b62e2e01bbdca87ab68a68ffae5e228217da5679
Author: AhyoungRyu 
Date:   2017-04-04T06:33:39Z

[ZEPPELIN-2341] Remove -Psparkr build profile

### What is this PR for?
Currently users who build Zeppelin from source need to include `-Psparkr` 
to use `%r` with embedded local Spark. But it's quite inconvenient to write 
this build profile every time we build i think. So I removed `-Psparkr` and 
make `r` related libraries automatically downloaded when we build Zeppelin like 
I did #2213

### What type of PR is it?
Improvement

### Todos
* [x] - remove the rest of `-Psparkr` build profile in 
`dev/create_release.sh`, `dev/publish_release.sh`, and `docs/install/build.md` 
after #2213 merged

### What is the Jira issue?
[ZEPPELIN-2341](https://issues.apache.org/jira/browse/ZEPPELIN-2341)

### How should this be tested?
1. Apply this patch
2. Build source with below command
```
mvn clean package -DskipTests -pl 'zeppelin-interpreter, zeppelin-zengine, 
zeppelin-server, zeppelin-display, spark, spark-dependencies'

```
Aftr this step, there will be `R` dir under 
`ZEPPELIN_HOME/interpreter/spark`. Before this PR, only `dep` dir and 
`zeppelin-spark_2.10-0.8.0-SNAPSHOT.jar` is generated without `-Psparkr` build 
profile.

4. Restart Zeppelin. To make sure, run R tutorial note under `Zeppelin 
Tutorial` folder

It should be run successfully without any error

### Screenshots (if appropriate)
If we build without `-Psparkr`
- before : R related properties are not activated by default in Spark 
interpreter
![screen shot 2017-04-03 at 4 31 49 
pm](https://cloud.githubusercontent.com/assets/10060731/24599560/b952e414-188b-11e7-80db-ac649c869c02.png)

 - after

![after](https://cloud.githubusercontent.com/assets/10060731/24599567/bc513a94-188b-11e7-9e93-7abca3428279.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: AhyoungRyu 
Author: Ahyoung Ryu 

Closes #2215 from AhyoungRyu/ZEPPELIN-2341/includeSparkRByDefault and 
squashes the following commits:

8db18cc [AhyoungRyu] Remove the rest of '-Psparkr' in docs & sh files
f891fd4 [Ahyoung Ryu] Merge branch 'master' into 
ZEPPELIN-2341/includeSparkRByDefault
445be3e [AhyoungRyu] Add SPARKR env to check each test case need to 
download r dep or not
67af02a [AhyoungRyu] Remove -PsparkR in travis file
a00466c [AhyoungRyu] Remove sparkr build profile in pom files

commit 32e86d02d8ea8fdc66a2c1a39bf5cc2317922556
Author: AhyoungRyu 
Date:   2017-04-04T06:47:21Z

[Minor] Fix wrong Python lib path in Matplotlib tutorial note

### What is this PR for?
There is a wrong information about Zeppelin’s matplotlib backend files. 
It actually located under `ZEPPELIN_HOME/interpreter/lib/python/*` but tutorial 
note says it as `ZEPPELIN_HOME/lib/python/*`. So I corrected it.

### What type of PR is it?
kind a typo fix

### What is the Jira issue?
N/A

### Screenshots (if appropriate)
 - Before
![screen shot 2017-04-04 at 3 50 06 
pm](https://cloud.githubusercontent.com/assets/10060731/24644505/e573d3a0-194e-11e7-826f-4022bc5e1572.png)

 - After
![screen shot 2017-04-04 at 3 50 56 
pm](https://cloud.githubusercontent.com/assets/10060731/24644507/e6ae25c2-194e-11e7-91d7-f42b327d820e.png)

### Questions:
* Does the licenses files need