[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884557639 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884558345 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884558345 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884559303 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884559303 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884561932 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/notebook/NoteManagerTest.java: ## @@ -158,4 +166,117 @@ public void testLruCache() throws IOException { assert

[GitHub] [zeppelin] Reamer commented on pull request #4369: [ZEPPELIN-5737] try to avoid deadlock

2022-05-30 Thread GitBox
Reamer commented on PR #4369: URL: https://github.com/apache/zeppelin/pull/4369#issuecomment-1140876273 If no other comments are received, I will merge this pull request with the master on Thursday. -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [zeppelin] asfgit closed pull request #4367: Small typos in Spark introduction

2022-05-30 Thread GitBox
asfgit closed pull request #4367: Small typos in Spark introduction URL: https://github.com/apache/zeppelin/pull/4367 -- 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 unsubsc

[GitHub] [zeppelin] asfgit closed pull request #4361: [ZEPPELIN-5682] SQL keywords are not highlighted in paragraph

2022-05-30 Thread GitBox
asfgit closed pull request #4361: [ZEPPELIN-5682] SQL keywords are not highlighted in paragraph URL: https://github.com/apache/zeppelin/pull/4361 -- 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] saLeox commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
saLeox commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884708245 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java: ## @@ -181,7 +182,9 @@ public void saveNote(Note note, AuthenticationInfo subject) thro

[GitHub] [zeppelin] saLeox commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
saLeox commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884708245 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java: ## @@ -181,7 +182,9 @@ public void saveNote(Note note, AuthenticationInfo subject) thro

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
Reamer commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884759936 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java: ## @@ -181,7 +182,9 @@ public void saveNote(Note note, AuthenticationInfo subject) thro

[GitHub] [zeppelin] saLeox commented on a diff in pull request #4374: [ZEPPELIN-5744] Allow NoteManager for concurrent operation

2022-05-30 Thread GitBox
saLeox commented on code in PR #4374: URL: https://github.com/apache/zeppelin/pull/4374#discussion_r884708245 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java: ## @@ -181,7 +182,9 @@ public void saveNote(Note note, AuthenticationInfo subject) thro

[jira] [Created] (ZEPPELIN-5746) Markdown should support escaping html

2022-05-30 Thread Peter Lee (Jira)
Peter Lee created ZEPPELIN-5746: --- Summary: Markdown should support escaping html Key: ZEPPELIN-5746 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5746 Project: Zeppelin Issue Type: Improv

[GitHub] [zeppelin] PeterAlfredLee opened a new pull request, #4375: ZEPPELIN-5746: Add support for escaping html in markdown

2022-05-30 Thread GitBox
PeterAlfredLee opened a new pull request, #4375: URL: https://github.com/apache/zeppelin/pull/4375 This is implemented by flexmark. A new configuration is also added, and this configuration is turned on by default. ### What is this PR for? Zeppelin should support escaping html in