[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 please merge this PR if there are no more comments --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 any exception then it will print them --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...
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 by swich on 'zeppelin' repository at https://travis-ci.org/profile and travis-ci. And then make sure 'Build branch updates' option is enabled in the settings https://travis-ci.org/SachinJanani/zeppelin/settings. To trigger CI after setup, you will need ammend your last commit with git commit --amend git push your-remote HEAD --force See http://zeppelin.apache.org/contribution/contributions.html#continuous-integration. ``` --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---