[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-23 Thread karth295
Github user karth295 commented on the issue:

https://github.com/apache/zeppelin/pull/2738
  
These two commits are rebased over master.


---


[GitHub] zeppelin pull request #2743: [HOTFIX] minor typo of ZEPPELIN-3181

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

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


---


[GitHub] zeppelin pull request #2743: [HOTFIX] minor typo of ZEPPELIN-3181

2018-01-23 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

[HOTFIX] minor typo of ZEPPELIN-3181

### What is this PR for?
Trivial change of typo. 


### What type of PR is it?
[Hot Fix]

### Todos
* [ ] - Task

### What is the Jira issue?


### How should this be tested?


### 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 HotFix_ZEPPELIN-3181

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

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


commit 06cbddc6da49d5e94b971ce3a8059dcff28cbd5d
Author: Jeff Zhang 
Date:   2018-01-24T04:42:54Z

[HOTFIX] minor typo of ZEPPELIN-3181




---


[GitHub] zeppelin pull request #2736: ZEPPELIN-3179. Improve error message when IPyth...

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

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


---


[GitHub] zeppelin pull request #2737: ZEPPELIN-3181. Enable impersonation support for...

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

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


---


[GitHub] zeppelin issue #2736: ZEPPELIN-3179. Improve error message when IPython is n...

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

https://github.com/apache/zeppelin/pull/2736
  
More if no more comments


---


[GitHub] zeppelin issue #2737: ZEPPELIN-3181. Enable impersonation support for kerber...

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

https://github.com/apache/zeppelin/pull/2737
  
Merge if no more comments


---


[jira] [Created] (ZEPPELIN-3188) Umbrella of ZEPPELIN HA

2018-01-23 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3188:


 Summary: Umbrella of ZEPPELIN HA
 Key: ZEPPELIN-3188
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3188
 Project: Zeppelin
  Issue Type: New Feature
Reporter: Jeff Zhang






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


[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

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

https://github.com/apache/zeppelin/pull/2738
  
@karth295 Please try to rebase your PR, I think the build issue you 
mentioned above has been fixed. 


---


[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-23 Thread karth295
Github user karth295 commented on the issue:

https://github.com/apache/zeppelin/pull/2738
  
The TravisCI failure doesn't look right to me:

```
[WARNING] 
Dependency convergence error for com.google.guava:guava:15.0 paths to 
dependency are:
+-org.apache.zeppelin:zeppelin-zengine:0.8.0-SNAPSHOT
  +-org.apache.zeppelin:zeppelin-interpreter:0.8.0-20170802.043121-7
+-com.google.guava:guava:15.0
and
+-org.apache.zeppelin:zeppelin-zengine:0.8.0-SNAPSHOT
  +-com.google.guava:guava:23.0-android
[WARNING] The requested profile "web-ci" could not be activated because it 
does not exist.
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-enforcer-plugin:1.3.1:enforce (enforce) on 
project zeppelin-zengine: 
org.apache.maven.plugins.enforcer.DependencyConvergence failed with message:
[ERROR] Failed while enforcing releasability the error(s) are [
[ERROR] Dependency convergence error for com.google.guava:guava:15.0 paths 
to dependency are:
[ERROR] +-org.apache.zeppelin:zeppelin-zengine:0.8.0-SNAPSHOT
[ERROR]   +-org.apache.zeppelin:zeppelin-interpreter:0.8.0-20170802.043121-7
[ERROR] +-com.google.guava:guava:15.0
[ERROR] and
[ERROR] +-org.apache.zeppelin:zeppelin-zengine:0.8.0-SNAPSHOT
[ERROR]   +-com.google.guava:guava:23.0-android
```

If you run `mvn clean package`, it should not use the cached 
`org.apache.zeppelin:zeppelin-interpreter:0.8.0-20170802.043121-7`, and the 
dependencies should be correct. (Though maven still scares/confuses me)


---


[GitHub] zeppelin issue #2742: [WIP] [ZEPPELIN-3168] Interpreter Settings Authorizati...

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

https://github.com/apache/zeppelin/pull/2742
  
If user create new interpreter property, is it readonly by default or not ?


---


[GitHub] zeppelin issue #2742: [ZEPPELIN-3168] Interpreter Settings Authorization

2018-01-23 Thread necosta
Github user necosta commented on the issue:

https://github.com/apache/zeppelin/pull/2742
  
Hi @zjffdu , I didn't create a design doc but I hope the 
interpreter/overview.md file helps. 
We can always improve the .md file of course.

In a nutshell, I tried to use as much functionality as it exists, 
leveraging `set permissions` and re-using interpreter.json for persistence.

As you can see in the screenshot above, I slightly changed the position and 
title of "New Interpreter" button so it's more recognizable for the end-user.

I tested locally with basic Shiro setup and disabling 
`zeppelin.anonymous.allowed`.

I don't think there are any breaking changes but more tests are needed to 
be sure.





---


[GitHub] zeppelin pull request #2742: [ZEPPELIN-3168] Interpreter Settings Authorizat...

2018-01-23 Thread necosta
GitHub user necosta opened a pull request:

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

[ZEPPELIN-3168] Interpreter Settings Authorization

### What is this PR for?
Attempts to introduce some control/authorization over interpreter settings.
Added 2 levels of permissions: Owner and Reader
Also added a new attribute to InterpreterProperty to prevent editing a 
value (UI only)


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

### Todos
* [ ] - Task

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

### How should this be tested?
* Test interpreter setting creation/update/deletion
* Test interpreter setting property creation/update/deletion
* Test interpreter settings API

### Screenshots (if appropriate)

![int_sett1](https://user-images.githubusercontent.com/26248959/35284003-95c9a592-0051-11e8-8835-0479449a82ed.png)


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


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

$ git pull https://github.com/nokia/zeppelin zeppelin3168

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

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


commit 918c1d7d457a587ef71e4b51909389b333a585b5
Author: Nelson Costa 
Date:   2018-01-23T15:11:53Z

[ZEPPELIN-3168] Interpreter Settings Authorization




---


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

2018-01-23 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-23 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 

[GitHub] zeppelin issue #2741: [ZEPPELIN-3187] Remove doubles of settings in zeppelin...

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

https://github.com/apache/zeppelin/pull/2741
  
LGTM, thanks @mebelousov 


---


[GitHub] zeppelin pull request #2741: [ZEPPELIN-3187] Remove doubles of settings in z...

2018-01-23 Thread mebelousov
GitHub user mebelousov opened a pull request:

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

[ZEPPELIN-3187] Remove doubles of settings in zeppelin-site.xml.template

### What is this PR for?
Remove doubles in zeppelin-site.xml.template

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

### What is the Jira issue?
* [ZEPPELIN-3187]

### How should this be tested?
* No need

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


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

$ git pull https://github.com/tinkoff-dwh/zeppelin ZEPPELIN-3187

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

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


commit 44d6d08ead3385da47d9576ff27de3eac6211c54
Author: mebelousov 
Date:   2018-01-23T12:39:00Z

Remove doubles of settings




---


[jira] [Created] (ZEPPELIN-3187) Remove doubles in zeppelin-site.xml.template

2018-01-23 Thread Maxim Belousov (JIRA)
Maxim Belousov created ZEPPELIN-3187:


 Summary: Remove doubles in zeppelin-site.xml.template
 Key: ZEPPELIN-3187
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3187
 Project: Zeppelin
  Issue Type: Task
Affects Versions: 0.8.0
Reporter: Maxim Belousov
 Fix For: 0.8.0


There are doubles in inactive settings. Need to remove.



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


[GitHub] zeppelin issue #2739: [ZEPPELIN-3183] Add date format support for line chart...

2018-01-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2739
  
Thanks for the review will merge this if no more discussion.


---


[jira] [Created] (ZEPPELIN-3186) zeppelin cannot config hadoop.rpc.protection

2018-01-23 Thread richard.lee (JIRA)
richard.lee created ZEPPELIN-3186:
-

 Summary: zeppelin cannot config hadoop.rpc.protection
 Key: ZEPPELIN-3186
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3186
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters
Affects Versions: 0.7.2
Reporter: richard.lee


When using zeppelin's  with a kerberozied hadoop/spark cluster,  spark 
interpreter cannot start with following exception:

 
{code:java}
INFO [2018-01-23 18:46:05,013] ({pool-2-thread-2} Logging.scala[logInfo]:54) - 
Attempting to login to the Kerberos using principal: hadoop/xxx_h...@hadoop.com 
and keytab: /etc/hadoop/hadoop.service.keytab INFO [2018-01-23 18:46:05,048] 
({pool-2-thread-2} RMProxy.java[createRMProxy]:98) - Connecting to 
ResourceManager at l-opsdev1.dev.p1/10.64.16.34:8032 WARN [2018-01-23 
18:46:11,927] ({pool-2-thread-2} Client.java[run]:678) - Couldn't setup 
connection for hadoop/xxx_h...@hadoop.com to kdc/10.64.xx.xx:8032 
javax.security.sasl.SaslException: No common protection layer between client 
and server at 
com.sun.security.sasl.gsskerb.GssKrb5Client.doFinalHandshake(GssKrb5Client.java:251)
 at 
com.sun.security.sasl.gsskerb.GssKrb5Client.evaluateChallenge(GssKrb5Client.java:186)
 at 
org.apache.hadoop.security.SaslRpcClient.saslEvaluateToken(SaslRpcClient.java:483)
 at 
org.apache.hadoop.security.SaslRpcClient.saslConnect(SaslRpcClient.java:427) at 
org.apache.hadoop.ipc.Client$Connection.setupSaslConnection(Client.java:560) at 
org.apache.hadoop.ipc.Client$Connection.access$1900(Client.java:375) at 
org.apache.hadoop.ipc.Client$Connection$2.run(Client.java:729) at 
org.apache.hadoop.ipc.Client$Connection$2.run(Client.java:725) at 
java.security.AccessController.doPrivileged(Native Method) at 
javax.security.auth.Subject.doAs(Subject.java:422) at 
org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
 at org.apache.hadoop.ipc.Client$Connection.setupIOstreams(Client.java:725) at 
org.apache.hadoop.ipc.Client$Connection.access$2900(Client.java:375) at 
org.apache.hadoop.ipc.Client.getConnection(Client.java:1528) at 
org.apache.hadoop.ipc.Client.call(Client.java:1451) at 
org.apache.hadoop.ipc.Client.call(Client.java:1412) at 
org.apache.hadoop.ipc.ProtobufRpcEngine$Invoker.invoke(ProtobufRpcEngine.java:229)
 at com.sun.proxy.$Proxy15.getClusterMetrics(Unknown Source) at 
org.apache.hadoop.yarn.api.impl.pb.client.ApplicationClientProtocolPBClientImpl.getClusterMetrics(ApplicationClientProtocolPBClientImpl.java:206)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:497) at 
org.apache.hadoop.io.retry.RetryInvocationHandler.invokeMethod(RetryInvocationHandler.java:191)
 at 
org.apache.hadoop.io.retry.RetryInvocationHandler.invoke(RetryInvocationHandler.java:102)
 at com.sun.proxy.$Proxy16.getClusterMetrics(Unknown Source) at 
org.apache.hadoop.yarn.client.api.impl.YarnClientImpl.getYarnClusterMetrics(YarnClientImpl.java:487)
 at org.apache.hadoop.ipc.Client$Connection.setupIOstreams(Client.java:725) at 
org.apache.hadoop.ipc.Client$Connection.access$2900(Client.java:375) at 
org.apache.hadoop.ipc.Client.getConnection(Client.java:1528) at 
org.apache.hadoop.ipc.Client.call(Client.java:1451) at 
org.apache.hadoop.ipc.Client.call(Client.java:1412) at 
org.apache.hadoop.ipc.ProtobufRpcEngine$Invoker.invoke(ProtobufRpcEngine.java:229)
 at com.sun.proxy.$Proxy15.getClusterMetrics(Unknown Source) at 
org.apache.hadoop.yarn.api.impl.pb.client.ApplicationClientProtocolPBClientImpl.getClusterMetrics(ApplicationClientProtocolPBClientImpl.java:206)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:497) at 
org.apache.hadoop.io.retry.RetryInvocationHandler.invokeMethod(RetryInvocationHandler.java:191)
 at 
org.apache.hadoop.io.retry.RetryInvocationHandler.invoke(RetryInvocationHandler.java:102)
 at com.sun.proxy.$Proxy16.getClusterMetrics(Unknown Source) at 
org.apache.hadoop.yarn.client.api.impl.YarnClientImpl.getYarnClusterMetrics(YarnClientImpl.java:487)
 at 
org.apache.spark.deploy.yarn.Client$$anonfun$submitApplication$1.apply(Client.scala:156)
 at 
org.apache.spark.deploy.yarn.Client$$anonfun$submitApplication$1.apply(Client.scala:156)
 at org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) at 
org.apache.spark.deploy.yarn.Client.logInfo(Client.scala:59) at 
org.apache.spark.deploy.yarn.Client.submitApplication(Client.scala:155) at 

[GitHub] zeppelin issue #2740: [ZEPPELIN-3185] Add profiles for adding dependencies f...

2018-01-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2740
  
@zjffdu  Please help review this.


---


[GitHub] zeppelin pull request #2740: [ZEPPELIN-3185] Add profiles for adding depende...

2018-01-23 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-3185] Add profiles for adding dependencies for Hive and Phoenix

### What is this PR for?
This PR adds profiles for adding dependencies for Hive and Phoenix

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

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


### 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/prabhjyotsingh/zeppelin ZEPPELIN-3185

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

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


commit 99eafdecb7b04440f0546adfdfd46f51e8581a41
Author: Prabhjyot Singh 
Date:   2018-01-23T10:55:59Z

add profile for adding dependencies for hive and phoenix

Change-Id: I3ddfa2af7b7f6742a49f615ba4af5f4cd06478ff




---