[GitHub] zeppelin pull request #1550: [ZEPPELIN-912] Apply new mechanism to GeodeOqlI...

2016-10-23 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1520: [ZEPPELIN-1061] Select default interpreter while creat...

2016-10-23 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1520
  
@jongyoul & @AhyoungRyu thanks for reviewing it. @AhyoungRyu I will look 
into the issue.
@corneadoug any updates on this?

Thanks,
Kavin
MailTo: kavin.ku...@imaginea.com


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1539: [ZEPPELIN-1567] Let JDBC interpreter use user credenti...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1539
  
it would be even better to let user chose interpreter from dropdown list of 
interpreters in place of `entity` field in credentials menu, what do you think 
@Leemoonsoo @astroshim ? Also need to rebase I guess


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1555: [MINOR] ui improvement

2016-10-23 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1555
  
Tested and LGTM. Merge if there is no more discussion.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1556: [MINOR] remove comments in NotebookRestApi.java

2016-10-23 Thread vectorijk
GitHub user vectorijk opened a pull request:

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

[MINOR] remove comments in NotebookRestApi.java

### What is this PR for?
Since ZEPPELIN-1162 (PR #1341) has been solved, we should remove TODO 
comments correspondingly.

### What type of PR is it?
Minor

### 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/vectorijk/zeppelin minor-remove-comments

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

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


commit 1e8ce8acf236a6d8bc30cfc83f7bd184b8ae65f7
Author: Kai Jiang 
Date:   2016-10-24T03:23:01Z

remove TODO comments since 1162 has been solved




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1540: [ZEPPELIN-1566] Make paragraph editable with double cl...

2016-10-23 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1540
  
It looks good to hide editor on md. And I'm not good at ui, but in case of 
angular, I think it's better to hide it, too.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-23 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1479
  
@hayssams Do you have any clue that you mentioned? I don't know why two 
different jars while building Zeppelin.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1460: [ZEPPELIN-1490] pass subject while persisting n...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at:

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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1460: [ZEPPELIN-1490] pass subject while persisting note on ...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1460
  
similar approach was used as a part of #1265 to address this issue and is 
merged already, closing this one


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1555: [MINOR] ui improvement

2016-10-23 Thread baekhoseok
GitHub user baekhoseok opened a pull request:

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

[MINOR] ui improvement

### What is this PR for?
* configuration sorting
* remove tooltip in job menu
* change repository infomation icon


### What type of PR is it?
Improvement

### Screenshots (if appropriate)
* configuration sorting

before

https://cloud.githubusercontent.com/assets/9584854/19631867/d50001f8-99db-11e6-81fc-47de15a3561b.png;>

after
https://cloud.githubusercontent.com/assets/9584854/19631896/1688513e-99dc-11e6-8a8a-b87b996858cd.png;>

* remove tooltip in job menu

before

https://cloud.githubusercontent.com/assets/9584854/19631909/2a5cd90a-99dc-11e6-92dd-9738924de361.png;>

after

https://cloud.githubusercontent.com/assets/9584854/19631914/4c4a7216-99dc-11e6-9e39-c08640c49923.png;>


* change repository information icon

before 

https://cloud.githubusercontent.com/assets/9584854/19631951/8d6b592c-99dc-11e6-8de1-28e760a16ebf.png;>

after
https://cloud.githubusercontent.com/assets/9584854/19631919/5407de6c-99dc-11e6-8043-a73669695114.png;>


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



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

$ git pull https://github.com/baekhoseok/zeppelin minor_ui_fix

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

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


commit cd4af65490b3f8b0768f989a53ed047b506dd52f
Author: baekhoseok 
Date:   2016-10-24T02:05:15Z

minor ui improvement

commit bf98a03981f6efc0114bd904a21309df29e5d652
Author: baekhoseok 
Date:   2016-10-24T02:14:41Z

minor ui improvement




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1537
  
LGTM merging if no more discussion 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-23 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1462
  
@felixcheung I think it is not necessary as I didn't find junit in 
https://github.com/apache/zeppelin/blob/master/zeppelin-distribution/src/bin_license/LICENSE


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-10-23 Thread mfelgamal
Github user mfelgamal commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@astroshim Thank you for your reviews, Can you check the docker image again?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1462: ZEPPELIN-1477. Add Integration Test for LivyInterprete...

2016-10-23 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1462
  
One question - I'm not sure if dependencies for test only code should be 
listed in LICENSE.
Otherwise LGTM.



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1576) Bump Flink to 1.1.3

2016-10-23 Thread Trevor Grant (JIRA)
Trevor Grant created ZEPPELIN-1576:
--

 Summary: Bump Flink to 1.1.3
 Key: ZEPPELIN-1576
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1576
 Project: Zeppelin
  Issue Type: Bug
Reporter: Trevor Grant
Priority: Minor


Apache Flink has released version 1.1.3



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-23 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@minahlee Thank you. I rebase it and solve conflicts now. Please confirm it!

[https://travis-ci.org/hyonzin/zeppelin](https://travis-ci.org/hyonzin/zeppelin)


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-23 Thread hayssams
Github user hayssams commented on the issue:

https://github.com/apache/zeppelin/pull/1479
  
@jongyoul @zjffdu 
This issue is with hadoop 2.7 only. With hadoop 2.6 making hadoop 
dependency as provided works well.


I get the following exception
Caused by: java.lang.NoSuchMethodError: 
org.apache.hadoop.tracing.SpanReceiverHost.get(Lorg/apache/hadoop/conf/Configuration;Ljava/lang/String;)Lorg/apache/hadoop/tracing/SpanReceiverHost;
at org.apache.hadoop.hdfs.DFSClient.(DFSClient.java:634)
at org.apache.hadoop.hdfs.DFSClient.(DFSClient.java:619)
at 
org.apache.hadoop.hdfs.DistributedFileSystem.initialize(DistributedFileSystem.java:149)
at 
org.apache.hadoop.fs.FileSystem.createFileSystem(FileSystem.java:2596)
at org.apache.hadoop.fs.FileSystem.access$200(FileSystem.java:91)
at 
org.apache.hadoop.fs.FileSystem$Cache.getInternal(FileSystem.java:2630)
at org.apache.hadoop.fs.FileSystem$Cache.get(FileSystem.java:2612)
at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:370)
at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:169)
at org.apache.zeppelin.util.HdfsSite.mkdirs(HdfsSite.java:98)



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1537
  
CI is green, ready for review


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1517: [MINOR] ui improvement

2016-10-23 Thread baekhoseok
Github user baekhoseok closed the pull request at:

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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1537: [Zeppelin-1561] Improve sync for multiuser envi...

2016-10-23 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request:

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

[Zeppelin-1561] Improve sync for multiuser environment 

### What is this PR for?
apply multi-tenancy for storage sync mechanism

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

### Todos
* [x] - broadcast on sync
* [x] - set permissions for pulled notes

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

### How should this be tested?
Outline the steps to test the PR here.

### Screenshots (if appropriate)
green CI

### 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/khalidhuseynov/incubator-zeppelin 
improve/sync-multiuser

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

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


commit 85d60e50f780c2a4d678cfdcdf82a429fa7b51c7
Author: Khalid Huseynov 
Date:   2016-09-22T06:16:03Z

don't sync on start

commit 1ee2448bed9774baf4aa682daf37def7dcf09b1f
Author: Khalid Huseynov 
Date:   2016-09-22T06:17:08Z

reload, sync and broadcast on login

commit 73c2e95e10c553f82b8af3625feb01ac059f9995
Author: Khalid Huseynov 
Date:   2016-09-23T02:35:08Z

set perms for pulling notes - make them private

commit d0a8c3c3c7eaf84e2796997bd85d3d565101550f
Author: Khalid Huseynov 
Date:   2016-10-18T11:13:16Z

add isAnonymous

commit 1c11c2ddbd27b222669278f22f87b65cd2292fc9
Author: Khalid Huseynov 
Date:   2016-10-19T08:05:26Z

remove syncOnStart

commit e41b9b9324971b472a9210b03c189f01ffc6a0f3
Author: Khalid Huseynov 
Date:   2016-10-23T11:35:18Z

address comment + test




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1537: [Zeppelin-1561] Improve sync for multiuser envi...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at:

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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1537: [Zeppelin-1561] Improve sync for multiuser environment...

2016-10-23 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1537
  
@Leemoonsoo your comment makes sense if there's already existing acl for 
the note. addressed it with test under e41b9b9324971b472a9210b03c189f01ffc6a0f3


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1554: [WIP] [ZEPPELIN-1320] Run zeppelin interpreter process...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1554
  
know issue: restart on interpreter does not work as expected.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1554: [ZEPPELIN-1320] Run zeppelin interpreter proces...

2016-10-23 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-1320] Run zeppelin interpreter process as web front end user

Have recreated this from https://github.com/apache/zeppelin/pull/1322

### What is this PR for?

While running a Notebook using shell, spark, python uses same user as which 
zeppelin server is running. Which means these interprets have same permission 
on file system as zeppelin server.
IMO users should be able to impersonate themselves as a complete security 
system.

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

### Todos
* [ ] - Update doc
* [x] - FIX NPEs
* [x] - FIX CI

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

### How should this be tested?
 - Enable shiro auth in shiro.ini
 - Add ssh key for the same user you want to try and impersonate (say 
user1).
```
adduser user1
ssh-keygen
ssh user1@localhost mkdir -p .ssh
cat ~/.ssh/id_rsa.pub | ssh user1@localhost 'cat >> .ssh/authorized_keys'
```
 - Start zeppelin server, try and run following in paragraph in a notebook
 - Go to interpreter setting page, and enable "User Impersonate" in any of 
the interpreter (in my example its shell interpreter)

```
%sh
whoami
```

Check that it should run as new user, i.e. "user1"

### Screenshots (if appropriate)

![user 
impersonate](https://cloud.githubusercontent.com/assets/674497/17700306/d9294e80-63e2-11e6-8be8-1c4b84ef5aee.gif)



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



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

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-1320-2

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

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


commit aff1bf0645785102bff90e99150d40d8bbc2f03d
Author: Prabhjyot Singh 
Date:   2016-10-23T07:46:59Z

user should have option to run these interpreters as different user.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-10-23 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@mfelgamal Thank you for contributing this. 
I have several questions. 

1. I think it needs `-it` option for running docker.
so The command to run zeppelin docker should be 
`docker run --rm -it -p 8080:8080 -p 8081:8081 
mahmoudelgamal/zeppelin-release:0.5.0 bash`


2. What do you think of adding documents for this like 
[this](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/spark_cluster_mode.html#spark-standalone-mode)
 ?


3. I got an following error when I run the `Zeppelin Tutorial`, Could you 
take care a look this?


![image](https://cloud.githubusercontent.com/assets/3348133/19624728/1278843a-993f-11e6-935f-99b46f83c069.png)



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1322: [ZEPPELIN-1320] Run zeppelin interpreter proces...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at:

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


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1322: [ZEPPELIN-1320] Run zeppelin interpreter process as we...

2016-10-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1322
  
Closing this, will open a new one with merge of 
https://github.com/apache/zeppelin/pull/1265.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-10-23 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@jongyoul I totally agree with you adding multi-tenancy but how about 
separating that to another PR because we also already have restarting 
interpreter function in the `Interpreter Setting page` . 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1553: [ZEPPELIN-1575] [WIP] Notebook Repo settings UI

2016-10-23 Thread anthonycorbacho
GitHub user anthonycorbacho opened a pull request:

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

[ZEPPELIN-1575] [WIP] Notebook Repo settings UI

### What is this PR for?
Let user configure/change notebook repo setting via Zeppelin UI
 We can imagine one case with LocalNotebookRepo where the user might want 
to change the notebook repo folder to another location for instance.

***NB***: this scope of the PR doesn't include save change in the 
configuration file of zeppelin as well as implementation of every notebook 
repo, git, s3 etcetc, this will be added later if needed. 


### What type of PR is it?
 * **Improvement**

### Todos
* [ ] - Implement backend.
* [ ] - Implement frontend.
* [ ] - Implement LocalVFS directory change as exemple.
* [ ] - Add Documentation.

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

### How should this be tested?
TBD

### Screenshots (if appropriate)
TBD

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


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

$ git pull https://github.com/anthonycorbacho/incubator-zeppelin 
feat/NotebookRepoSettings

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

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


commit 667da3e869134c4ed8b11ee1356279f6a615ab7b
Author: Anthony Corbacho 
Date:   2016-10-23T04:56:08Z

Add new Rest api endpoint for notebook Repo settings

commit 17474f8f7ca9b8b48af562bdc903cdc7bd236044
Author: Anthony Corbacho 
Date:   2016-10-23T05:02:40Z

Added utility class for notebook re po payload deserialization

commit 602eb0cb04f3d4db2f751d6a95f5f3fe13455148
Author: Anthony Corbacho 
Date:   2016-10-23T05:03:40Z

Update method :: updateRepoSettings to create NotebookRepoSettingsRequest 
with Empty static object instead of null

commit 9c28c4b0643d6b25cb9e5f3675d6bc270110497b
Author: Anthony Corbacho 
Date:   2016-10-23T05:11:47Z

Update zeppelin server and add NotebookRepo rest api to the singleton

commit fd5199a9cd74250fad29b769c6ff98f3beba5216
Author: Damien CORNEAU 
Date:   2016-10-11T08:35:52Z

Base structure for notebook repo ui

(cherry picked from commit 6109a933daf931a50aab4352c9903b0b2db3c3c0)

commit fef76174431f70b217cb24ce94246f041151615b
Author: Damien CORNEAU 
Date:   2016-10-11T10:58:22Z

Render mocked list of notebook repo settings

(cherry picked from commit 26cce46d9fe6e305317fc479b49c577b5f217bb8)

commit b74b43ab7f58f5500d7e18352957bff441abf601
Author: Damien CORNEAU 
Date:   2016-10-12T02:37:16Z

Change presentation of notebook settings

(cherry picked from commit ba9f59bdf049a406592f721d9030d2abc8cb79b8)

commit c5c6e574c5014884ccad6d2995bc17925621684e
Author: Damien CORNEAU 
Date:   2016-10-12T04:21:03Z

Get Repos from api

(cherry picked from commit 8753e05ce43187436d125d4a2a69e1744aa2e69f)

commit bf3b38878e2e3c027340de1108130037d1cde8fe
Author: Damien CORNEAU 
Date:   2016-10-16T05:28:57Z

Implement edit

(cherry picked from commit 2e09b851b019a5361ab45820a7dbd443b14cbe5a)

commit 003faa558afae169b126d3e2b42f949e69d7cde7
Author: Damien CORNEAU 
Date:   2016-10-16T07:41:11Z

Improve edit repo

(cherry picked from commit 37ba504aaf5fb3500cbfe1634b0722ea9107c72d)

commit fd36cd6ae7f0cd03041ca1d73d422adab120c891
Author: Damien CORNEAU 
Date:   2016-10-16T07:51:36Z

Improve ux

(cherry picked from commit f8ddfc10414f0027e5c2202839e1cdeea50d1831)

commit 02d002deaca9f83bfc104d3ab62c6f1935715594
Author: Anthony Corbacho 
Date:   2016-10-23T05:27:52Z

Added new wrapper class

commit 68dde90832b5622ef367156608bd0b104836d43b
Author: Anthony Corbacho 
Date:   2016-10-23T05:34:05Z

Add new methods in notebookRepo :: get and set settings in notebook

commit 3637f887a4afa813d852e8189240213cf1ab62d6
Author: Anthony Corbacho 
Date:   2016-10-23T05:36:54Z

Add new wrapper for notebook repo PLUS settings

commit 464065387e6f0b8adf7c9131ed57d895fc3e5bab
Author: Anthony Corbacho 
Date:   2016-10-23T05:40:15Z

Added 2 new methos in notebookRepoSync, exposed get and update notebook repo

commit 9d6cf3a2f843d283f6f08c44f8e1a8a37076d9d0
Author: Anthony Corbacho 

[jira] [Created] (ZEPPELIN-1575) Notebook Repo settings UI

2016-10-23 Thread Anthony Corbacho (JIRA)
Anthony Corbacho created ZEPPELIN-1575:
--

 Summary: Notebook Repo settings UI
 Key: ZEPPELIN-1575
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1575
 Project: Zeppelin
  Issue Type: Improvement
  Components: rest-api, zeppelin-zengine
Reporter: Anthony Corbacho
Assignee: Anthony Corbacho
 Fix For: 0.7.0


provide a way to let user configure his notebook repo (Local storage, git 
etcetc) via Zeppelin UI.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)