GitHub user wary opened a pull request:
https://github.com/apache/zeppelin/pull/2612
remove the duplicate property username in RemoteInterpreter
### What is this PR for?
* Remove the username property in class RemoteInterpreter, instead reuse
the username property in class Interpreterï¼ to avoid get null value with
method getUserName in the super class Interpreter
* Overwrite username get/set method in class LazyOpenInterpreter
### What type of PR is it?
[Improvement]
### Todos
* [ ] - Task
### What is the Jira issue?
* [ZEPPELIN-2981]
### How should this be tested?
* First time? Setup Travis CI as described on
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed
behavior
* Outline any manual steps to test the PR here.
### 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/wary/zeppelin RemioteInterpreterUserNameFIx
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2612.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 #2612
----
commit 4af0253b84fe393e465abdb655501767cbe20e55
Author: lei wang <[email protected]>
Date: 2017-10-09T14:48:00Z
remove the duplicate property username
commit 356101d2033f04cc2430c526e3f236e885c609bb
Author: lei wang <[email protected]>
Date: 2017-10-09T15:15:56Z
fix the username property in LazyOpenInterpreter
----
---
