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 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
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 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
GitHub user SachinJanani opened a pull request:
https://github.com/apache/zeppelin/pull/2518
Fixing zeppelin-2281 by clearing output when data is available
### What is this PR for?
Fixing ZEPPELIN-2281 by clearing previous when new data is available
### What type
Github user SachinJanani closed the pull request at:
https://github.com/apache/zeppelin/pull/1234
---
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
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1234
@jongyoul Sure I will close it now.Thanks
---
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
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1234
@jongyoul I revised the PR title and its description
---
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 user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1234
Travis checks are failing but are not related to this changes.@jongyoul Can
you please have a look.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1234
Failure of travis check is not related to this change
---
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 user SachinJanani opened a pull request:
https://github.com/apache/zeppelin/pull/1234
[ZEPPELIN-1234] Fix for bug ZEPPELIN-1234
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
Thanks @minahlee
---
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
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
@jongyoul This changes are only for branch-0.6 as master does not have this
files (even this code) anymore due to helium changes.Also i think issue might
not occur for zeppelin-0.7.So
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
Can someone please merge this PR as I don't have permission
---
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 user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
Thanks @jongyoul. Sure will make the change also will add a comment in the
code about about the 30 seconds timeout
---
If your project is set up for it, you can reply to this email and have
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
@jongyoul Yes you are right if RemoteInterpreterServer fails to launch
within 10 seconds then this test will fail,but we need to have some threshold
for that because if say there is some
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1172
@corneadoug Will this changes also be available in 0.6.1 ?
---
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 user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
Can someone please review this PR
---
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
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1197
One of the travis check is failing but its not related to my changes
---
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 user SachinJanani opened a pull request:
https://github.com/apache/zeppelin/pull/1197
[ZEPPELIN-1196] Fix for bug ZEPPELIN-1196
### What is this PR for?
Fixed issue related to connecting to remote running interpreter process
with multiple interpreters in interpreter
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/955
Thanks @Leemoonsoo @jongyoul @AhyoungRyu @corneadoug For reviewing.Can
someone please merge this pull request?
---
If your project is set up for it, you can reply to this email and have your
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/955
@jongyoul @corneadoug Done with the docs can you please have a look.
---
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 user SachinJanani commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/955
@jongyoul Yes you are right.But this changes will allow user to connect to
existing remote process instead of zeppelin running the interpreter.I think we
should create a separate
Github user SachinJanani commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/955
@jongyoul There can be scenarios where user is starting RemoteInterpreter
embedded in his service by calling
`RemoteInterpreterServer server = new RemoteInterpreterServer(port
Github user SachinJanani commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/955
@Leemoonsoo @jongyoul One of the TravisCI task is failing but I think its
not related to my changes, is there any way I can rerun the travisCI without
pushing changes
Github user SachinJanani commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/955
@Leemoonsoo @jongyoul This is what will look now:
