[GitHub] [zeppelin] zjffdu commented on pull request #4298: [ZEPPELIN-5660] Implement move gcsnotebookrepo folder

2022-03-01 Thread GitBox
zjffdu commented on pull request #4298: URL: https://github.com/apache/zeppelin/pull/4298#issuecomment-1055502643 LGTM -- 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 u

[GitHub] [zeppelin] huage1994 commented on pull request #4299: [ZEPPELIN-5661] Support clone a specific revision of a note

2022-03-01 Thread GitBox
huage1994 commented on pull request #4299: URL: https://github.com/apache/zeppelin/pull/4299#issuecomment-1055271116 @zjffdu Thanks a lot for the reviews. And I have modified the code to keep a good code style. -- This is an automated message from the Apache Git Service. To respond to t

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4299: [ZEPPELIN-5661] Support clone a specific revision of a note

2022-03-01 Thread GitBox
zjffdu commented on a change in pull request #4299: URL: https://github.com/apache/zeppelin/pull/4299#discussion_r816619334 ## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java ## @@ -310,25 +311,40 @@ public String importNote(String sourceJs

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4299: [ZEPPELIN-5661] Support clone a specific revision of a note

2022-03-01 Thread GitBox
zjffdu commented on a change in pull request #4299: URL: https://github.com/apache/zeppelin/pull/4299#discussion_r816615868 ## File path: zeppelin-server/src/test/java/org/apache/zeppelin/rest/NotebookRestApiTest.java ## @@ -695,33 +693,69 @@ public void testRunAllParagraph_Fi

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4299: [ZEPPELIN-5661] Support clone a specific revision of a note

2022-03-01 Thread GitBox
zjffdu commented on a change in pull request #4299: URL: https://github.com/apache/zeppelin/pull/4299#discussion_r816611568 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java ## @@ -317,15 +317,30 @@ public void renameNote(String noteI

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4299: [ZEPPELIN-5661] Support clone a specific revision of a note

2022-03-01 Thread GitBox
zjffdu commented on a change in pull request #4299: URL: https://github.com/apache/zeppelin/pull/4299#discussion_r816608156 ## File path: docs/usage/rest_api/notebook.md ## @@ -352,7 +352,8 @@ Notebooks REST API supports the following operations: List, Create, Get, Delete,

[GitHub] [zeppelin] Reamer commented on a change in pull request #4286: [ZEPPELIN-5613] zeppelin-interpreter-parent: Add dependencyManagement for log4j2 in pom.xml

2022-03-01 Thread GitBox
Reamer commented on a change in pull request #4286: URL: https://github.com/apache/zeppelin/pull/4286#discussion_r816580090 ## File path: zeppelin-interpreter-parent/pom.xml ## @@ -33,6 +33,35 @@ 0.9.0-SNAPSHOT Zeppelin: Interpreter Parent + +2.17.1 + + + +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4288: [ZEPPELIN-5632] zeppelin-zengine: Skip load notebook when it is damaged.

2022-03-01 Thread GitBox
zjffdu commented on a change in pull request #4288: URL: https://github.com/apache/zeppelin/pull/4288#discussion_r816549086 ## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java ## @@ -78,8 +78,8 @@ private void init() throws IOException {

[ANNOUNCE] ZEPPELIN 0.10.1 is released

2022-03-01 Thread Jeff Zhang
Hi Folks, The Apache Zeppelin community is pleased to announce the availability of the 0.10.1 release. Zeppelin is a collaborative data analytics and visualization tool for distributed, general-purpose data processing system such as Apache Spark, Apache Flink, Apache Cassandra and etc. This is m

[GitHub] [zeppelin] Mabin-J commented on a change in pull request #4288: [ZEPPELIN-5632] zeppelin-zengine: Skip load notebook when it is damaged.

2022-03-01 Thread GitBox
Mabin-J commented on a change in pull request #4288: URL: https://github.com/apache/zeppelin/pull/4288#discussion_r816537601 ## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java ## @@ -78,8 +78,8 @@ private void init() throws IOException {

[GitHub] [zeppelin] Mabin-J commented on a change in pull request #4288: [ZEPPELIN-5632] zeppelin-zengine: Skip load notebook when it is damaged.

2022-03-01 Thread GitBox
Mabin-J commented on a change in pull request #4288: URL: https://github.com/apache/zeppelin/pull/4288#discussion_r816537601 ## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java ## @@ -78,8 +78,8 @@ private void init() throws IOException {

[GitHub] [zeppelin] Mabin-J commented on a change in pull request #4288: [ZEPPELIN-5632] zeppelin-zengine: Skip load notebook when it is damaged.

2022-03-01 Thread GitBox
Mabin-J commented on a change in pull request #4288: URL: https://github.com/apache/zeppelin/pull/4288#discussion_r816537601 ## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/NoteManager.java ## @@ -78,8 +78,8 @@ private void init() throws IOException {