[GitHub] zeppelin pull request #2756: [ZEPPELIN-3200] Move R temporary dir(R/lib) int...

2018-01-31 Thread jwagun
GitHub user jwagun opened a pull request:

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

[ZEPPELIN-3200] Move R temporary dir(R/lib) into r

### What is this PR for?
Move R LIB_DIR into subdirectory of zeppelin/r

### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3200


### How should this be tested?
* "mvn package -Pr" and check the R install params

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/jwagun/zeppelin ZEPPELIN-3200

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

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


commit e46b24f1c2852eff383464bf5befa5840c756016
Author: Changhyup Jwa 
Date:   2018-01-31T02:55:16Z

Move R temporary dir(R/lib) into r




---


[GitHub] zeppelin pull request #2755: [ZEPPELIN-3200] Move R LIB_DIR into r

2018-01-31 Thread jwagun
Github user jwagun closed the pull request at:

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


---


[GitHub] zeppelin pull request #2755: Move R LIB_DIR into r

2018-01-31 Thread jwagun
GitHub user jwagun opened a pull request:

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

Move R LIB_DIR into r

### What is this PR for?
Move R LIB_DIR into subdirectory of zeppelin/r

### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3200

### How should this be tested?
 - "mvn package -Pr" and check the R install params

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/jwagun/zeppelin ZEPPELIN-3200

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

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


commit e46b24f1c2852eff383464bf5befa5840c756016
Author: Changhyup Jwa 
Date:   2018-01-31T02:55:16Z

Move R temporary dir(R/lib) into r




---


[jira] [Created] (ZEPPELIN-3200) Update R LIB_DIR from "zeppelin/R" to "zepplin/r/R"

2018-01-31 Thread Changhyup Jwa (JIRA)
Changhyup Jwa created ZEPPELIN-3200:
---

 Summary: Update R LIB_DIR from "zeppelin/R" to "zepplin/r/R"
 Key: ZEPPELIN-3200
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3200
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Changhyup Jwa


When I try to package zeppelin with R, R installation params are as follows.

+ FWDIR=/home/jwagun/opensource/zeppelin/r/R
+ LIB_DIR=/home/jwagun/opensource/+*zeppelin/r/R/../../R/lib*+

As "zeppelin/r/R/../../R/lib" means "zeppelin/R/lib", "zeppelin/R" is created.
In my opinion, it should be changed as follows.

+ FWDIR=/home/jwagun/opensource/zeppelin/r/R
+ LIB_DIR=/home/jwagun/opensource/+*zeppelin/r/R/lib*+



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (ZEPPELIN-3199) empty python file generated when running sparkr

2018-01-31 Thread Arun M J (JIRA)
Arun M J created ZEPPELIN-3199:
--

 Summary: empty python file generated when running sparkr
 Key: ZEPPELIN-3199
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3199
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.2
Reporter: Arun M J


while running a sparkr paragraph/notebook  an unwanted empty python file is 
generated in temporary directory along with R script file.

```

[root@node9 zeppelin]# ll /tmp/|grep zepp
-rw-r--r-- 1 root root 12203 Jan 31 13:06 
zeppelin_pyspark-4295577785768557857.py
-rw-r--r-- 1 root root 0 Jan 31 14:01 zeppelin_pyspark-4687233077932682453.py
-rw-r--r-- 1 root root 0 Feb 1 07:50 zeppelin_pyspark-4787864373810914127.py
-rw-r--r-- 1 root root 12203 Jan 31 21:16 
zeppelin_pyspark-5109236865481070078.py
-rw-r--r-- 1 root root 0 Jan 31 13:10 zeppelin_pyspark-6974344656378923535.py
-rw-r--r-- 1 root root 12203 Feb 1 07:46 zeppelin_pyspark-8682464468211230492.py
-rw-r--r-- 1 root root 4101 Jan 31 13:10 zeppelin_sparkr-3440271701865396258.R
-rw-r--r-- 1 root root 4101 Feb 1 07:50 zeppelin_sparkr-6794813169689699636.R
-rw-r--r-- 1 root root 4101 Jan 31 14:01 zeppelin_sparkr-8692405291837295226.R

```

i wonder , whether this an implementation result or a bug.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin issue #2709: ZEPPELIN-3111. Refactor SparkInterpreter

2018-01-31 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2709
  
Thanks @Leemoonsoo , will merge it if no more comments


---


[jira] [Created] (ZEPPELIN-3198) UI should not show Version/GIT Control if the same if not supported

2018-01-31 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-3198:
-

 Summary: UI should not show Version/GIT Control if the same if not 
supported 
 Key: ZEPPELIN-3198
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3198
 Project: Zeppelin
  Issue Type: Bug
Reporter: Prabhjyot Singh


Currently, UI shows an option for version/GIT Control even when it is not 
supported by the underlying implementing storage configuration. 

It is only after users try to save a commit and get an error "Couldn't 
checkpoint note revision: possibly storage doesn't support versioning. Please 
check the logs for more details.".

So, if implementing storage configuration doesn't support git storage, UI 
should not show those options.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin issue #2709: ZEPPELIN-3111. Refactor SparkInterpreter

2018-01-31 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2709
  
Thanks @zjffdu for explain. Looks good to me.


---


[GitHub] zeppelin issue #2754: Update configuration.md

2018-01-31 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2754
  
Thanks for the fix. LGTM


---


[GitHub] zeppelin pull request #2745: [ZEPPELIN-3189] NPE on paragraph run via API

2018-01-31 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[jira] [Created] (ZEPPELIN-3197) Dynamic Interpreter Loading/Unloading results in 404

2018-01-31 Thread Simeon H.K. Fitch (JIRA)
Simeon H.K. Fitch created ZEPPELIN-3197:
---

 Summary: Dynamic Interpreter Loading/Unloading results in 404
 Key: ZEPPELIN-3197
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3197
 Project: Zeppelin
  Issue Type: Bug
  Components: documentation, Interpreters, zeppelin-interpreter, 
zeppelin-server
Affects Versions: 0.7.3
Reporter: Simeon H.K. Fitch


According to the documentation, one should be able to dynamically load and 
unload Zeppelin interpreters at runtime:

http://zeppelin.apache.org/docs/0.7.3/manual/dynamicinterpreterload.html

However, going to the `/api/interpreter/load/` and `/api/interpreter/unload/` 
REST endpoints results in a 404 HTTP response.

Looking through the source, I'm unable to find where the `load` and `unload` 
endpoints might be handled. Therefore, either a) the docs or incorrect, or b) 
the feature isn't working.




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin pull request #2753: ZEPPELIN-3138. checkstyle for zeppelin-interpre...

2018-01-31 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[GitHub] zeppelin pull request #2754: Update configuration.md

2018-01-31 Thread skymon
GitHub user skymon opened a pull request:

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

Update configuration.md

Corrected spelling "ba" -> "be"

### What is this PR for?
Minor spelling correction


### What type of PR is it?
Documentation 

### Todos
-

### What is the Jira issue
-

### How should this be tested?
-

### Screenshots (if appropriate)

### Questions:
-


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

$ git pull https://github.com/skymon/zeppelin patch-1

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

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


commit c4a7be75e0fdfe637cf25dc81afc936c340cf474
Author: skymon 
Date:   2018-01-31T10:54:53Z

Update configuration.md

Corrected spelling "ba" -> "be"




---


[GitHub] zeppelin pull request #2700: [ZEPPELIN-3092] GitHub Integration

2018-01-31 Thread mohamagdy
Github user mohamagdy closed the pull request at:

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


---


[GitHub] zeppelin pull request #2700: [ZEPPELIN-3092] GitHub Integration

2018-01-31 Thread mohamagdy
GitHub user mohamagdy reopened a pull request:

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

[ZEPPELIN-3092] GitHub Integration

### What is this PR for?
GitHub integration as a storage for notebooks.


### What type of PR is it?
Feature

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

### How should this be tested?
1. Change the configuration in `zeppelin-site.xml` to enable GitHub 
integration (add GitHub url, username, access token and origin) as described in 
https://github.com/apache/zeppelin/compare/master...mohamagdy:zeppelin-3092-remote-github-integration?expand=1#diff-89104d48f0358450399a6f679bba9c4f
2. Start the Zeppelin server
3. Open an existing notebook or create a new notebook
4. Do some changes to the notebook, for example add a new paragraph
5. Click on the versioning button on the top menu to commit and save changes
6. Checkout the changes in the GitHub repository. The changes should be 
reflected 

### Questions:
* **Does the licenses files need update?**
No

* **Is there breaking changes for older versions?**
No

* **Does this needs documentation?**
Yes. Documentation is updated as part of the pull request.


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

$ git pull https://github.com/mohamagdy/zeppelin 
zeppelin-3092-remote-github-integration

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

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


commit 94675032cce563e169889d411e0d420ac30f3a49
Author: Mohamed Magdy 
Date:   2017-12-05T13:12:10Z

[ZEPPELIN-3092] Add `zepplein-server/local-repo` to `.gitignore`

commit 0bde310138666dadb69006ee586c8312ded8e148
Author: Mohamed Magdy 
Date:   2017-12-05T13:12:49Z

[ZEPPELIN-3092] Add `zeppelin-site.xml` to `zeppelin-server` resources

commit eeb485a2fec19f80c0043fb6053fec71c5a83d41
Author: Mohamed Magdy 
Date:   2017-12-05T13:13:41Z

[ZEPPELIN-3092] Add Github configuration reader

commit 32f6764b19562720453c0930ad7335c07d3b29e4
Author: Mohamed Magdy 
Date:   2017-12-05T16:52:50Z

[ZEPPELIN-3092] Fix GitNotebook test

- Setting the proper path for the test notebooks
- Add the test notebooks to the `src/test/resource` directory

commit 33ae24a473188be4ca81a79456703287f2fe07a5
Author: Mohamed Magdy 
Date:   2017-12-05T16:54:42Z

[ZEPPELIN-3092] Add remote Github repository synchronzing

- Pull the latest changes when the notebook page loads in the browser
- After commiting the changes, pull the latest changes from the remote repo 
then push the latest changes to the remote repo

commit bb0afe2fca3f7915bd3f6047fdc8860d29722ba6
Author: Mohamed Magdy 
Date:   2017-12-08T16:33:17Z

[ZEPPELIN-3092] Add GitHub remote to configurations

commit 2dbf11622cadc200b7d4d06c215ff0b5fa2c39b1
Author: Mohamed Magdy 
Date:   2017-12-08T17:03:23Z

[ZEPPELIN-3092] Add GitHub configuration to `zeppelin-site.xml` template

commit 5236176ad9cf157aaf30886eaf43f50d537f8510
Author: Mohamed Magdy 
Date:   2017-12-08T17:04:11Z

[ZEPPELIN-3092] Move GitHub notebook repostiory to separte file

commit 843e42a2736af325fe2240e5f8d03e2ec0c5d8f4
Author: Mohamed Magdy 
Date:   2017-12-08T17:24:19Z

[ZEPPELIN-3092] Cleanup GitHub repository tests

commit 2b093b2056bbff1fd8595f3db11549c23f02d534
Author: Mohamed Magdy 
Date:   2017-12-08T17:40:40Z

[ZEPPELIN-3092] Add documentation about GitHub integration

commit 0dacbf1c2fb8bb7013af7545cb173cb9a30ad058
Author: Mohamed Magdy 
Date:   2017-12-08T17:46:46Z

[ZEPPELIN-3092] Fix encoding in the documenation

commit aadd9b5f36dcd8f6ef21cadc08d2da36f1e1902b
Author: Mohamed Magdy 
Date:   2017-12-08T17:50:49Z

[ZEPPELIN-3092] Revert back ZeppelinServer changes

The changes were removed because it was closing the local server when any 
key is
pressed in the console.

commit b77a2d33706bf33bbe698c3328c1bb8ed7a5f1da
Author: Mohamed Magdy 
Date:   2017-12-08T17:53:34Z

[ZEPPELIN-3092] Fix identation in `pom.xml`

commit 6aa4ba7ddd8c0055643e5ae8d487cedf35f0de8e
Author: Mohamed Magdy 
Date:   2017-12-08T17:56:04Z

[ZEPPELIN-3092] Revert back `GitNotebookRepo` to `master`

Only change the visiblity of the `getGit` method to `protected`

commit 3009abd47de3048cae922c82e36081d61608f3e9
Author: Mohamed Magdy 
Date:   2017-12-08T18:03:28Z

[ZEPPELIN-3092] Reset `GitNotebookRepo` to `master`

commit 81969e123a4fb8e28501262e3b12f2380951f45e
Author: Mohamed Magdy 
Date:   2017-12-08T18:06:40Z

[ZEPPELIN-3092] Add documentation for loading notebooks from repo

The change is about refreshing the notebooks everytime the repository is 
requested. Refreshing means,
fetching th