You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by minahlee <gi...@git.apache.org> on 2016/08/11 13:35:07 UTC

[GitHub] zeppelin pull request #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

GitHub user minahlee opened a pull request:

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

    [ZEPPELIN-1287][branch-0.6] No need to call print to display output in PythonInterpreter

    ### What is this PR for?
    Implement #1278 to merge branch-0.6
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    [ZEPPELIN-1287](https://issues.apache.org/jira/browse/ZEPPELIN-1287)
    
    ### 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/minahlee/zeppelin branch-0.6_ZEPPELIN-1287

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

    https://github.com/apache/zeppelin/pull/1320.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 #1320
    
----
commit ac83b14d880b1b6fa2586349cb4d69c7f8c1db2e
Author: Mina Lee <mi...@apache.org>
Date:   2016-08-11T13:30:11Z

    No need to call print to display output in PythonInterpreter

----


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

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

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


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

Posted by minahlee <gi...@git.apache.org>.
GitHub user minahlee reopened a pull request:

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

    [ZEPPELIN-1287][branch-0.6] No need to call print to display output in PythonInterpreter

    ### What is this PR for?
    Implement #1278 to merge branch-0.6
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    [ZEPPELIN-1287](https://issues.apache.org/jira/browse/ZEPPELIN-1287)
    
    ### 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/minahlee/zeppelin branch-0.6_ZEPPELIN-1287

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

    https://github.com/apache/zeppelin/pull/1320.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 #1320
    
----
commit ac83b14d880b1b6fa2586349cb4d69c7f8c1db2e
Author: Mina Lee <mi...@apache.org>
Date:   2016-08-11T13:30:11Z

    No need to call print to display output in PythonInterpreter

----


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call print to d...

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

    https://github.com/apache/zeppelin/pull/1320
  
    @bzz Thanks for taking care of JIRA issue!


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call print to d...

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

    https://github.com/apache/zeppelin/pull/1320
  
    @zjffdu Thanks for review!
    Merge if there is no more discussion


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

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

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


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call print to d...

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

    https://github.com/apache/zeppelin/pull/1320
  
    Just one minor code style issue, rest LGTM, Thanks @minahlee for taking over this. 


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

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

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


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call print to d...

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

    https://github.com/apache/zeppelin/pull/1320
  
    \cc @zjffdu for review


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call pri...

Posted by minahlee <gi...@git.apache.org>.
GitHub user minahlee reopened a pull request:

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

    [ZEPPELIN-1287][branch-0.6] No need to call print to display output in PythonInterpreter

    ### What is this PR for?
    Implement #1278 to merge branch-0.6
    
    ### What type of PR is it?
    Bug Fix
    
    ### What is the Jira issue?
    [ZEPPELIN-1287](https://issues.apache.org/jira/browse/ZEPPELIN-1287)
    
    ### 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/minahlee/zeppelin branch-0.6_ZEPPELIN-1287

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

    https://github.com/apache/zeppelin/pull/1320.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 #1320
    
----
commit ac83b14d880b1b6fa2586349cb4d69c7f8c1db2e
Author: Mina Lee <mi...@apache.org>
Date:   2016-08-11T13:30:11Z

    No need to call print to display output in PythonInterpreter

----


---
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 #1320: [ZEPPELIN-1287][branch-0.6] No need to call print to d...

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

    https://github.com/apache/zeppelin/pull/1320
  
    @minahlee I think we also need to set appropriate "fix for" versions in JIRA before merging #1278  otherwise it will be very hard for you, as release manager to figure out changelog and attributions may get lost.
    
    I have updated current one to both `0.7.0` for master and `0.6.1` for branch-0.6
    
    LGTM


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