[GitHub] zeppelin pull request #2612: remove the duplicate property username in Remot...

[wary]

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 <lei1...@outlook.com>
Date:   2017-10-09T14:48:00Z

    remove the duplicate property username

commit 356101d2033f04cc2430c526e3f236e885c609bb
Author: lei wang <lei1...@outlook.com>
Date:   2017-10-09T15:15:56Z

    fix the username property in LazyOpenInterpreter

----


---