[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2518 Thanks @SachinJanani for the fix! Merge to master and branch-0.7 if no further comments. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-17 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/2518 @Leemoonsoo Merging the upstream and restarting the travis build did the trick.Here is the travis link for this PR-https://travis-ci.org/SachinJanani/zeppelin/builds/265444667.Can you

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-14 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/2518 Yes @Leemoonsoo I have tried restarting failed job but it didn't succeed :( --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2518 Not sure why CI is failing. @SachinJanani Have you tried restart failed job? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-09 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/2518 This build is failing due to some environment issue in jenkins can someone please help me out. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-09 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/2518 @Leemoonsoo Thanks for the review.Have started the travis build. @felixcheung With this change it will not clear the output if the execution fails and keep the previous output.But if there

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2518 Thanks @SachinJanani for contribution. Could you try setup CI and see it this PR passes the CI build? ``` Looks like travis-ci is not configured for your fork. Please setup