[GitHub] zeppelin pull request #2207: [Zeppelin-802] Support for Zeppelin Context red...

[cloverhearts]

GitHub user cloverhearts opened a pull request:

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

    [Zeppelin-802] Support for Zeppelin Context redefinition on Python and 
Pyspark 

    ### What is this PR for?
    If you override the reserved word ZeppelinContext such as `z` in the python 
language, the whole paragraph output problem occurred.
    I have taken care to avoid this issue.
    
    `z` == `_zc` == zeppelin context
    
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-802
    
    ### How should this be tested?
    The error should not occur in the following situations:
    ```
    %python
    z = 1
    print("Hello Zeppelin")
    ```
    
    ```
    %pyspark
    z = 1
    print("Hello Zeppelin")
    ```
    
    
    ### Screenshots (if appropriate)
    
    #### before
    ![replace zeppelin 
context-err](https://cloud.githubusercontent.com/assets/10525473/24521772/319946be-15c8-11e7-96cf-7fdf41c70a66.png)
    
    #### after
    ![replace zeppelin 
context](https://cloud.githubusercontent.com/assets/10525473/24521775/349fa7cc-15c8-11e7-8fe4-4f3f5597deff.png)
    
    ### 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/cloverhearts/zeppelin 
ZEPPELIN-802-pyspark-zeppelin-context

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

    https://github.com/apache/zeppelin/pull/2207.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 #2207
    
----
commit 7edb1ad4196f4d9e9791545ad25980727ad418db
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:26:07Z

    replace name zeppelin context on pyspark

commit 2d0ea1a030109243a7d530d0e88793859b3f9a14
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:35:28Z

    added test code

commit b54209da0bd60ce0ae1e46a41ed1695e5fa442b8
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:44:53Z

    fix pyspark test case

commit 0eaf9a896c30037dd4e51dc18df0a2c5b9abf69c
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:46:08Z

    replace name zeppelin context on python

commit f3805b3b73c533fc8dabcc39c15dfcecae0259a4
Author: CloverHearts <cloverhearts...@gmail.com>
Date:   2017-03-30T18:46:27Z

    added test case on python

----


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