[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-20 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2021 CI was passing here. merged to master and 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

[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-18 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2021 @prabhjyotsingh @felixcheung Thanks for review, the comments are addressed. --- 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

[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2021 I think that's a fair ask - @zjffdu could you update and we will merge? --- 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

[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-16 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2021 Ran this couple of time on my local, works a expected. --- 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

[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2021 LGTM Waiting for CI and merging if no more comment --- 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

[GitHub] zeppelin issue #2021: [ZEPPELIN-2116]. livy interpreter needs to restart aft...

2017-02-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2021 @prabhjyotsingh @felixcheung Please help review, CI failure is not relevant. ``` TypeError: namedtuple() missing 3 required keyword-only arguments: 'verbose', 'rename', and 'module'