You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by cloverhearts <gi...@git.apache.org> on 2017/04/17 13:44:33 UTC

[GitHub] zeppelin pull request #2257: [Branch-0.7] Fixes a build problem in Zeppelin ...

GitHub user cloverhearts opened a pull request:

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

    [Branch-0.7] Fixes a build problem in Zeppelin branch0.7.

    Fixes a build problem in Zeppelin branch0.7.
    Hotfix.

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

    $ git pull https://github.com/cloverhearts/zeppelin fix/branch-0.7-build-hotfix

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

    https://github.com/apache/zeppelin/pull/2257.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 #2257
    
----
commit a3a615c3bf0be92089d03105b0fad96a05b66940
Author: CloverHearts <cl...@gmail.com>
Date:   2017-04-12T08:14:36Z

    [Zeppelin-802] Support for Zeppelin Context redefinition on Python and Pyspark
    
    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`
    
    Improvement
    
    https://issues.apache.org/jira/browse/ZEPPELIN-802
    
    The error should not occur in the following situations:
    ```
    %python
    z = 1
    print("Hello Zeppelin")
    ```
    
    ```
    %pyspark
    z = 1
    print("Hello Zeppelin")
    ```
    
    ![replace zeppelin context-err](https://cloud.githubusercontent.com/assets/10525473/24521772/319946be-15c8-11e7-96cf-7fdf41c70a66.png)
    
    ![replace zeppelin context](https://cloud.githubusercontent.com/assets/10525473/24521775/349fa7cc-15c8-11e7-8fe4-4f3f5597deff.png)
    
    * Does the licenses files need update? no
    * Is there breaking changes for older versions? no
    * Does this needs documentation? no
    
    Author: CloverHearts <cl...@gmail.com>
    Author: cloverhearts <cl...@gmail.com>
    
    Closes #2207 from cloverhearts/ZEPPELIN-802-pyspark-zeppelin-context and squashes the following commits:
    
    cc986010 [CloverHearts] added completion on namespace
    14695cb8 [CloverHearts] Recovering a member name that is not associated with a namespace
    31af92ab [CloverHearts] fix test case _zc to __zeppelin__
    6697d677 [CloverHearts] apply to namespace and replace name _zc to __zeppelin__
    ca795cff [cloverhearts] replace output
    1d372df4 [cloverhearts] change name logger
    4e8435ac [CloverHearts] added test case on python
    b6b804ad [CloverHearts] replace name zeppelin context on python
    9fbf70d6 [CloverHearts] fix pyspark test case
    987e2118 [CloverHearts] added test code
    5da3d6ed [CloverHearts] replace name zeppelin context on pyspark
    
    (cherry picked from commit 8d03920b9bec86dd8e2fc343d32749a08f501362)
    Signed-off-by: CloverHearts <cl...@gmail.com>

commit 2da0b423bb3075d6b960d2f8dbb5466f5f11b733
Author: CloverHearts <cl...@gmail.com>
Date:   2017-04-17T13:39:08Z

    fix branch-0.7 build error

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2257: [Branch-0.7] Support for Zeppelin Context redef...

Posted by cloverhearts <gi...@git.apache.org>.
Github user cloverhearts closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2257: [Branch-0.7] Support for Zeppelin Context redefinition...

Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/2257
  
    Merged. Please close this PR manually


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2257: [Branch-0.7] Support for Zeppelin Context redefinition...

Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/2257
  
    I'll merge it as hotfix into branch-0.7


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---