[GitHub] zeppelin pull request #1679: ZEPPELIN-1707. Pass userName when creating inte...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1679: ZEPPELIN-1707. Pass userName when creating inte...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1707. Pass userName when creating interpreter through thrift

### What is this PR for?
In ZEPPELIN-1607, I'd like refactor livy interpreter to scoped mode by 
default, this require username when open this interpreter. So I propose to pass 
username when creating interpreter through thrift.
What I did in this PR.
* update `RemoteInterpreterService.thrift` and regenerate the java thrift 
code.
* update `genthrift.sh`, otherwise hashCode method won't be generated 
correctly.
* This is one compilation issue (`PythonDockerInterpreterTest.java`) in the 
existing master branch, I also fix it here. 


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

### Todos
* [ ] - Task

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

### 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-1707

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

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


commit cbb829cc3754ce9090939c3deebe304300ef44f8
Author: Jeff Zhang 
Date:   2016-11-25T01:58:15Z

ZEPPELIN-1707. Pass userName when creating interpreter through thrift

commit e8e913fd85f2b627e096a3eeb1840af7157113e9
Author: Jeff Zhang 
Date:   2016-11-25T08:33:39Z

regenerate it using thrift 0.9.2




---
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 #1679: ZEPPELIN-1707. Pass userName when creating inte...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1679: ZEPPELIN-1707. Pass userName when creating inte...

[zjffdu]

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1707. Pass userName when creating interpreter through thrift

### What is this PR for?
In ZEPPELIN-1607, I'd like refactor livy interpreter to scoped mode by 
default, this require username when open this interpreter. So I propose to pass 
username when creating interpreter through thrift.
What I did in this PR.
* update RemoteInterpreterService.thrift and regenerate the java thrift 
code.
* I use thrift 0.9.2 because I notice in zeppelin/pom.xml, the 
libthrift.version we use is 0.9.2
* update genthrift.sh, otherwise hashCode method won't be generated 
correctly.


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

### Todos
* [ ] - Task

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

### 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-1707

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

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


commit 223a48f0ce8d0d768bc2eb0364b82e25f7a633ff
Author: Jeff Zhang 
Date:   2016-11-25T01:32:14Z

ZEPPELIN-1707. Pass userName when creating interpreter through thrift




---
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.
---