[GitHub] [zeppelin] jongyoul commented on pull request #4385: [ZEPPELIN-5434] Upgrade jetty version to 9.4.43.v20210629

2022-07-03 Thread GitBox
jongyoul commented on PR #4385: URL: https://github.com/apache/zeppelin/pull/4385#issuecomment-1173033520 @zholoda Yes, correct. I saw the 9.x version will be deprecated fully. So I would like to upgrade to the latest version of 11.x if possible. Could you please help to do it? -- This i

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4382: [ZEPPELIN-5754] Init Notes in background

2022-07-03 Thread GitBox
jongyoul commented on code in PR #4382: URL: https://github.com/apache/zeppelin/pull/4382#discussion_r912501438 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java: ## @@ -109,6 +119,60 @@ public void recoveryIfNecessary() { } } + /** + * Su

[GitHub] [zeppelin] jongyoul commented on pull request #4382: [ZEPPELIN-5754] Init Notes in background

2022-07-03 Thread GitBox
jongyoul commented on PR #4382: URL: https://github.com/apache/zeppelin/pull/4382#issuecomment-1173110653 Overall, LGTM except for the minor 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

[GitHub] [zeppelin] jongyoul merged pull request #4391: [ZEPPELIN-5724] Provide config to disable version control.

2022-07-03 Thread GitBox
jongyoul merged PR #4391: URL: https://github.com/apache/zeppelin/pull/4391 -- 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...@zeppelin.apa