[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-30 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1658 @agoodm merged master and addressed tests added from #1632. CI failure is not related. Merge to master. ![image](https://cloud.githubusercontent.com/assets/1540981/20757762/7087d81e

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-29 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1658 @Leemoonsoo FYI, #1632 added another set of tests. The interpreter result format for these should be changed as well. --- If your project is set up for it, you can reply to this email and have you

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1658 Thanks @agoodm @khalidhuseynov for the review! Merge to master if there're no more comments. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1658 tried locally and works as expected, 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 thi

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-28 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1658 Very nice @Leemoonsoo 👍 I have tested this with the matplotlib tutorial notebook just to see if [ZEPPELIN-1638](https://issues.apache.org/jira/browse/ZEPPELIN-1638) is also resolved by

[GitHub] zeppelin issue #1658: [ZEPPELIN-212] Multiple paragraph results

2016-11-25 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1658 I think this PR is ready to be reviewed. While this PR includes quite a lot of changes i'm highlighting some key changes. 1. Display multiple types of output in a single paragraph.