[GitHub] [zeppelin] zjffdu opened a new pull request #4227: [ZEPPELIN-5526] interpreter setting still in error status when remove the invalid dependency

2021-09-18 Thread GitBox
zjffdu opened a new pull request #4227: URL: https://github.com/apache/zeppelin/pull/4227 ### What is this PR for? The root cause is that the status and errorReason is not reset after invalid dependency is removed. This PR just reset them and also add unit test for that.

[GitHub] [zeppelin] zjffdu commented on pull request #4226: [ZEPPELIN-5524] Some configuration of flink interpreter are incorrect

2021-09-18 Thread GitBox
zjffdu commented on pull request #4226: URL: https://github.com/apache/zeppelin/pull/4226#issuecomment-922253038 Will merge if no more comment -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [zeppelin] jongyoul commented on pull request #4219: [WIP] Fix flaky test

2021-09-18 Thread GitBox
jongyoul commented on pull request #4219: URL: https://github.com/apache/zeppelin/pull/4219#issuecomment-922253340 May we merge this one? I think it helps to check the status of CI. :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [zeppelin] zjffdu commented on pull request #4219: [WIP] Fix flaky test

2021-09-18 Thread GitBox
zjffdu commented on pull request #4219: URL: https://github.com/apache/zeppelin/pull/4219#issuecomment-922253939 @jongyoul Not yet, it is not real green, I just ignore the error in this PR -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [zeppelin] jongyoul commented on pull request #4219: [WIP] Fix flaky test

2021-09-18 Thread GitBox
jongyoul commented on pull request #4219: URL: https://github.com/apache/zeppelin/pull/4219#issuecomment-922255373 Oh.. I didn't check the `core.yml` carefully. I see. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[jira] [Created] (ZEPPELIN-5527) Remove the dependency of `zeppelin-markdown` from `zeppelin-jupyter`

2021-09-18 Thread Jongyoul Lee (Jira)
Jongyoul Lee created ZEPPELIN-5527: -- Summary: Remove the dependency of `zeppelin-markdown` from `zeppelin-jupyter` Key: ZEPPELIN-5527 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5527 Project:

[GitHub] [zeppelin] jongyoul commented on a change in pull request #4223: [WIP] [ZEPPELIN-5518] Adopt JSR356 for Websocket

2021-09-18 Thread GitBox
jongyoul commented on a change in pull request #4223: URL: https://github.com/apache/zeppelin/pull/4223#discussion_r711572935 ## File path: zeppelin-jupyter/src/main/java/org/apache/zeppelin/jupyter/JupyterUtil.java ## @@ -78,7 +78,7 @@ public JupyterUtil() { .registe

[GitHub] [zeppelin] zjffdu commented on pull request #4216: [ZEPPELIN-5510]There are some logic problems, which may lead to thread leaks.

2021-09-18 Thread GitBox
zjffdu commented on pull request #4216: URL: https://github.com/apache/zeppelin/pull/4216#issuecomment-922407413 ping @NicolasGong -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

[GitHub] [zeppelin] asfgit closed pull request #4226: [ZEPPELIN-5524] Some configuration of flink interpreter are incorrect

2021-09-18 Thread GitBox
asfgit closed pull request #4226: URL: https://github.com/apache/zeppelin/pull/4226 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr..