[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808824577 ## File path: zeppelin-interpreter-integration/pom.xml ## @@ -203,4 +172,105 @@ + + + hadoop2 + + +true + + +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808836041 ## File path: zeppelin-interpreter/pom.xml ## @@ -233,4 +226,86 @@ + + + hadoop2 + + +true + + +$

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808838468 ## File path: zeppelin-server/pom.xml ## @@ -293,30 +293,9 @@ - org.apache.hadoop - hadoop-client - - - -

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808841411 ## File path: zeppelin-server/pom.xml ## @@ -498,17 +477,95 @@ + + + hadoop2 + + +${hadoop2.7.version} +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808842127 ## File path: zeppelin-zengine/pom.xml ## @@ -317,4 +312,86 @@ + + + + hadoop2 + + +true + + + +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808842804 ## File path: zeppelin-zengine/pom.xml ## @@ -317,4 +312,86 @@ + + + + hadoop2 + + +true + + + +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808844992 ## File path: zeppelin-zengine/pom.xml ## @@ -317,4 +312,86 @@ + + + + hadoop2 + + +true + + + +

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4266: [ZEPPELIN-5576] Improve Zeppelin CI

2022-02-17 Thread GitBox
zjffdu commented on a change in pull request #4266: URL: https://github.com/apache/zeppelin/pull/4266#discussion_r808846728 ## File path: zeppelin-zengine/pom.xml ## @@ -317,4 +312,86 @@ + + + + hadoop2 + + +true + + + +

[jira] [Created] (ZEPPELIN-5654) Implement Move for GCSNotebookRepo

2022-02-17 Thread Bagus Nugroho Budi Nurtomo (Jira)
Bagus Nugroho Budi Nurtomo created ZEPPELIN-5654: Summary: Implement Move for GCSNotebookRepo Key: ZEPPELIN-5654 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5654 Project: Zeppel

[GitHub] [zeppelin] BagusThanatos opened a new pull request #4297: Implement GCSNotebookRepo Move

2022-02-17 Thread GitBox
BagusThanatos opened a new pull request #4297: URL: https://github.com/apache/zeppelin/pull/4297 ### What is this PR for? This is to implement move function that is used for renaming notebooks and moving it to trash. Currently it would result in a no op, making users unable to rename

[GitHub] [zeppelin] zjffdu commented on pull request #4297: [ZEPPELIN-5654] Implement GCSNotebookRepo Move

2022-02-17 Thread GitBox
zjffdu commented on pull request #4297: URL: https://github.com/apache/zeppelin/pull/4297#issuecomment-1043736851 Thanks for the contribution, @BagusThanatos Have you ever tested it in google cloud? -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [zeppelin] BagusThanatos commented on pull request #4297: [ZEPPELIN-5654] Implement GCSNotebookRepo Move

2022-02-17 Thread GitBox
BagusThanatos commented on pull request #4297: URL: https://github.com/apache/zeppelin/pull/4297#issuecomment-1043743453 @zjffdu Yep We're currently using a custom jar with this patch in our Zeppelin instance. -- This is an automated message from the Apache Git Service. To resp

[GitHub] [zeppelin] zjffdu commented on pull request #4297: [ZEPPELIN-5654] Implement GCSNotebookRepo Move

2022-02-17 Thread GitBox
zjffdu commented on pull request #4297: URL: https://github.com/apache/zeppelin/pull/4297#issuecomment-1043755237 Awesome,@BagusThanatos PR LGTM, The CI is unstable recently, we are working it, will rerun the CI after the CI stable issue is fixed. -- This is an automated message from the

[GitHub] [zeppelin] zjffdu edited a comment on pull request #4297: [ZEPPELIN-5654] Implement GCSNotebookRepo Move

2022-02-17 Thread GitBox
zjffdu edited a comment on pull request #4297: URL: https://github.com/apache/zeppelin/pull/4297#issuecomment-1043755237 Awesome,@BagusThanatos PR LGTM, The CI is unstable recently, we are working on it, will rerun the CI after the CI stable issue is fixed. -- This is an automated messag

[jira] [Created] (ZEPPELIN-5655) OSSNotebookRepo support version control.

2022-02-17 Thread Guanhua LI (Jira)
Guanhua LI created ZEPPELIN-5655: Summary: OSSNotebookRepo support version control. Key: ZEPPELIN-5655 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5655 Project: Zeppelin Issue Type: N