Github user khalidhuseynov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-160115088
Thanks for the fix, 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 proj
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/468
---
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 fea
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-160054268
Merging if there isn't another discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-159809789
Ready to 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 project does not have
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-159801295
Looks good to me
---
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 th
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-159800609
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 this f
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-159780695
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 this fe
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/468#issuecomment-159191051
I've changed restart timing and checked the status of paragraphs. The test
passed if at least one of paragraphs has Status.ABORT.
---
If your project is se
GitHub user jongyoul opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/468
ZEPPELIN-458 Fix testAbortParagraphStatusOnInterpreterRestart
- Changed how to check some paragraph is aborted
- See the https://issues.apache.org/jira/browse/ZEPPELIN-458 for more