[GitHub] [zeppelin] jongyoul merged pull request #4407: [ZEPPELIN-5769] Roles in API-Request

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

[GitHub] [zeppelin] jongyoul merged pull request #4412: [ZEPPELIN-5770] Remove hazelcast jet interpreter

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

[GitHub] [zeppelin] jongyoul merged pull request #4418: [ZEPPELIN-5777] Fix dead links in docs

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

[GitHub] [zeppelin] Reamer commented on pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-22 Thread GitBox
Reamer commented on PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#issuecomment-1192219156 I will merge this next Monday if no further comments are received. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-21 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1192186454 Will merge it soon. -- 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

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-21 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1192186332 @ganlute Thank you for the detailed explanation. It makes sense to me. :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [zeppelin] zjffdu commented on pull request #4352: Bump hadoop-common from 2.6.5 to 3.2.3 in /flink/flink-scala-parent

2022-07-21 Thread GitBox
zjffdu commented on PR #4352: URL: https://github.com/apache/zeppelin/pull/4352#issuecomment-1192162837 CI is failed -- 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

[GitHub] [zeppelin] jongyoul commented on pull request #4352: Bump hadoop-common from 2.6.5 to 3.2.3 in /flink/flink-scala-parent

2022-07-21 Thread GitBox
jongyoul commented on PR #4352: URL: https://github.com/apache/zeppelin/pull/4352#issuecomment-1192162185 @zjffdu Could you please check this issue? I think it could be merged but I think you may be a proper reviewer. -- This is an automated message from the Apache Git Service. To

[GitHub] [zeppelin] jongyoul merged pull request #4387: Bump kotlin-stdlib from 1.3.50 to 1.6.0 in /kotlin

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

[GitHub] [zeppelin] jongyoul commented on pull request #4373: Bump grunt from 0.4.5 to 1.5.3 in /zeppelin-web

2022-07-21 Thread GitBox
jongyoul commented on PR #4373: URL: https://github.com/apache/zeppelin/pull/4373#issuecomment-1192160700 @dependabot rebase -- 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.

[GitHub] [zeppelin] jongyoul commented on pull request #4399: Bump commons-configuration2 from 2.7 to 2.8.0

2022-07-21 Thread GitBox
jongyoul commented on PR #4399: URL: https://github.com/apache/zeppelin/pull/4399#issuecomment-1192160173 @dependabot recreate -- 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

[GitHub] [zeppelin] jongyoul merged pull request #4411: Bump kubernetes-client from 5.3.1 to 5.3.2 in /zeppelin-plugins/launcher/k8s-standard

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

[GitHub] [zeppelin] jongyoul merged pull request #4413: Bump aws-java-sdk-s3 from 1.11.736 to 1.12.261 in /zeppelin-plugins/notebookrepo/s3

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

[GitHub] [zeppelin] dntjr8096 opened a new pull request, #4418: [ZEPPELIN-5777] Fix dead links in docs

2022-07-21 Thread GitBox
dntjr8096 opened a new pull request, #4418: URL: https://github.com/apache/zeppelin/pull/4418 ### What is this PR for? Most broken links have been updated with links from the latest documentation. However, in the case of cdh.md and cron_schduler.md, I modified it in a different way.

[GitHub] [zeppelin] dntjr8096 closed pull request #4417: [ZEPPELIN-5777] Fix dead links in docs

2022-07-21 Thread GitBox
dntjr8096 closed pull request #4417: [ZEPPELIN-5777] Fix dead links in docs URL: https://github.com/apache/zeppelin/pull/4417 -- 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.

[GitHub] [zeppelin] dntjr8096 opened a new pull request, #4417: [ZEPPELIN-5777] Fix dead links in docs

2022-07-21 Thread GitBox
dntjr8096 opened a new pull request, #4417: URL: https://github.com/apache/zeppelin/pull/4417 ### What is this PR for? Most broken links have been updated to the latest. However, in the case of cdh.md and cron_schduler.md, I fixed it by another method. In the case of cdh.md, the

[GitHub] [zeppelin] zjffdu commented on pull request #4388: [ZEPPELIN-5755] Support Spark 3.3

2022-07-20 Thread GitBox
zjffdu commented on PR #4388: URL: https://github.com/apache/zeppelin/pull/4388#issuecomment-1189963579 @shakraz Spark 3.3 has not been released when Zeppelin 0.10.1 is released -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [zeppelin] shakraz commented on pull request #4388: [ZEPPELIN-5755] Support Spark 3.3

2022-07-20 Thread GitBox
shakraz commented on PR #4388: URL: https://github.com/apache/zeppelin/pull/4388#issuecomment-1189954989 hey, zjffdu! quick question, not sure why but seems like the latest docker image (apache/zeppelin:0.10.1) doesn't support spark3.3, is it because it's simply not up to date?

[GitHub] [zeppelin] JoWonHyeung commented on pull request #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class

2022-07-20 Thread GitBox
JoWonHyeung commented on PR #4416: URL: https://github.com/apache/zeppelin/pull/4416#issuecomment-1189906438 Yes, An error occurred in the test and I am looking into it. I will upload it again later. Thank you! -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] zjffdu commented on pull request #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class

2022-07-20 Thread GitBox
zjffdu commented on PR #4416: URL: https://github.com/apache/zeppelin/pull/4416#issuecomment-1189885511 @JoWonHyeung Do you plan to create another PR? -- 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

[GitHub] [zeppelin] JoWonHyeung closed pull request #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class

2022-07-19 Thread GitBox
JoWonHyeung closed pull request #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class URL: https://github.com/apache/zeppelin/pull/4416 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [zeppelin] zjffdu commented on a diff in pull request #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class

2022-07-19 Thread GitBox
zjffdu commented on code in PR #4416: URL: https://github.com/apache/zeppelin/pull/4416#discussion_r924310745 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -327,7 +324,7 @@ public String getServerContextPath() { public

[GitHub] [zeppelin] JoWonHyeung opened a new pull request, #4416: [ZEPPELIN-5773]Fix isWindowsPath Function in ZeppelinConfiguration Class

2022-07-19 Thread GitBox
JoWonHyeung opened a new pull request, #4416: URL: https://github.com/apache/zeppelin/pull/4416 ### What is this PR for? Even though the user's operating system is windows, 'isWindowsPath' function returns False and enters the else. Therefore, I think that I need to change

[GitHub] [zeppelin] zjffdu commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-19 Thread GitBox
zjffdu commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1188775009 @zlosim How about updating

[GitHub] [zeppelin] zlosim commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-19 Thread GitBox
zlosim commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1188746965 Hi @zjffdu thanks for the response. To be honest, I think this fix is doing exactly what you are saying you want to achieve. Right now setting e.g. [spark dependencies via zeppelin

[GitHub] [zeppelin] zjffdu merged pull request #4404: [ZEPPELIN-5761] Support flink 1.15.1

2022-07-18 Thread GitBox
zjffdu merged PR #4404: URL: https://github.com/apache/zeppelin/pull/4404 -- 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:

[GitHub] [zeppelin] zjffdu commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-18 Thread GitBox
zjffdu commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1188531385 @zlosim

[GitHub] [zeppelin] zjffdu commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-18 Thread GitBox
zjffdu commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1188523244 @zlosim

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-18 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1188512777 > Yes, I just want to know the reason why you delete some - complex - tests. Could you please tell me the reason why you delete them instead of modifying them? In fact, the

[GitHub] [zeppelin] Reamer closed pull request #4415: Create Apache1

2022-07-18 Thread GitBox
Reamer closed pull request #4415: Create Apache1 URL: https://github.com/apache/zeppelin/pull/4415 -- 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:

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-18 Thread GitBox
Reamer commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r923374722 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message; -import

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-18 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1187287631 Yes, I just want to know the reason why you delete some - complex - tests. Could you please tell me the reason why you delete them instead of modifying them? -- This is an automated

[GitHub] [zeppelin] Reamer commented on pull request #4415: Create Apache1

2022-07-18 Thread GitBox
Reamer commented on PR #4415: URL: https://github.com/apache/zeppelin/pull/4415#issuecomment-1187200150 :question: -- 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

[GitHub] [zeppelin] NoorImann closed pull request #4415: Create Apache1

2022-07-18 Thread GitBox
NoorImann closed pull request #4415: Create Apache1 URL: https://github.com/apache/zeppelin/pull/4415 -- 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:

[GitHub] [zeppelin] NoorImann opened a new pull request, #4415: Create Apache1

2022-07-18 Thread GitBox
NoorImann opened a new pull request, #4415: URL: https://github.com/apache/zeppelin/pull/4415 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-17 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186778939 > LGTM but is it fine to remove some tests? We encourage adding tests so I recommend keeping tests if it's available. WDYT? Yes, I agree with you. I only keep the basic test cases

[GitHub] [zeppelin] jongyoul commented on pull request #4377: Bump @angular/core from 8.2.12 to 11.0.5 in /zeppelin-web-angular

2022-07-17 Thread GitBox
jongyoul commented on PR #4377: URL: https://github.com/apache/zeppelin/pull/4377#issuecomment-1186722608 @dependabot rebase -- 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.

[GitHub] [zeppelin] jongyoul commented on pull request #4397: Bump shiro-core from 1.8.0 to 1.9.1

2022-07-17 Thread GitBox
jongyoul commented on PR #4397: URL: https://github.com/apache/zeppelin/pull/4397#issuecomment-1186721161 @dependabot rebase -- 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.

[GitHub] [zeppelin] jongyoul merged pull request #4409: [ZEPPELIN-3945] Create .gitattributes

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

[GitHub] [zeppelin] dependabot[bot] closed pull request #3508: Bump eslint from 3.19.0 to 4.18.2 in /zeppelin-web

2022-07-17 Thread GitBox
dependabot[bot] closed pull request #3508: Bump eslint from 3.19.0 to 4.18.2 in /zeppelin-web URL: https://github.com/apache/zeppelin/pull/3508 -- 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] dependabot[bot] commented on pull request #3508: Bump eslint from 3.19.0 to 4.18.2 in /zeppelin-web

2022-07-17 Thread GitBox
dependabot[bot] commented on PR #3508: URL: https://github.com/apache/zeppelin/pull/3508#issuecomment-1186718823 Looks like eslint is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [zeppelin] jongyoul merged pull request #4414: [ZEPPELIN-5767] Update rimraf and scrollmonitor to the latest version

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

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-17 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186704587 hi @jongyoul , It is unbelievable that the CI runs failed again. I run action workflows twice in my repository(https://github.com/ganlute/zeppelin) and they work

[GitHub] [zeppelin] zjffdu commented on pull request #4404: [ZEPPELIN-5761] Support flink 1.15.1

2022-07-17 Thread GitBox
zjffdu commented on PR #4404: URL: https://github.com/apache/zeppelin/pull/4404#issuecomment-1186697268 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 specific

[GitHub] [zeppelin] NoorImann commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-17 Thread GitBox
NoorImann commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186647358 > 1,2,3,4,5,6 -- 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

[GitHub] [zeppelin] NoorImann commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-17 Thread GitBox
NoorImann commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186647214 > ### What is this PR for? > Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0 > > ### What type of PR is it? > Improvement > > ### Todos > * [ ] - Task

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-16 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186234093 > Yes, you need a permission. ~~Could you please give me your confluence ID? I'll give it to you. ~~ I gave you the permission baed on the ID which likes the wiki page similar to your

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-16 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186221035 Yes, you need a permission. Could you please give me your confluence ID? I'll give it to you. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-16 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186215409 > Thank you for the update!!! BTW, we are checking the status of interpreters so could you please update the wiki of

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-16 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186164728 Thank you for the update!!! BTW, we are checking the status of interpreters so could you please update the wiki of

[GitHub] [zeppelin] HYEEWON opened a new pull request, #4414: [ZEPPELIN-5767] Update rimraf and scrollmonitor to the latest version

2022-07-16 Thread GitBox
HYEEWON opened a new pull request, #4414: URL: https://github.com/apache/zeppelin/pull/4414 ### What is this PR for? * Update `rimraf` and `scrollmonitor` to the latest version and fix errors that occur while building on Windows ### What type of PR is it? Bug Fix ###

[GitHub] [zeppelin] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-15 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1186078299 https://user-images.githubusercontent.com/77536119/179337394-73c37c87-4b9b-49b9-b322-8ab8fb91d590.png;> hi @jongyoul , I pass CI loaclly, can you help to approve running workflows?

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4413: Bump aws-java-sdk-s3 from 1.11.736 to 1.12.261 in /zeppelin-plugins/notebookrepo/s3

2022-07-15 Thread GitBox
dependabot[bot] opened a new pull request, #4413: URL: https://github.com/apache/zeppelin/pull/4413 Bumps [aws-java-sdk-s3](https://github.com/aws/aws-sdk-java) from 1.11.736 to 1.12.261. Changelog Sourced from

[GitHub] [zeppelin] jinwoo-Bae opened a new pull request, #4412: [ZEPPELIN-5770] Remove hazelcast jet interpreter

2022-07-15 Thread GitBox
jinwoo-Bae opened a new pull request, #4412: URL: https://github.com/apache/zeppelin/pull/4412 ### What is this PR for? Remove an interpreter that has not been updated anymore since 2018. ### What type of PR is it? Refactoring ### Todos * [x] - Remove hazelcast

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-15 Thread GitBox
jongyoul commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r921898336 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message;

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-15 Thread GitBox
jongyoul commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r921898336 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message;

[GitHub] [zeppelin] jongyoul merged pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

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

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-15 Thread GitBox
Reamer commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r921875340 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message; -import

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4411: Bump kubernetes-client from 5.3.1 to 5.3.2 in /zeppelin-plugins/launcher/k8s-standard

2022-07-15 Thread GitBox
dependabot[bot] opened a new pull request, #4411: URL: https://github.com/apache/zeppelin/pull/4411 Bumps [kubernetes-client](https://github.com/fabric8io/kubernetes-client) from 5.3.1 to 5.3.2. Release notes Sourced from

[GitHub] [zeppelin] Reamer merged pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop3 profile to true for successful build.

2022-07-15 Thread GitBox
Reamer merged PR #4403: URL: https://github.com/apache/zeppelin/pull/4403 -- 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:

[GitHub] [zeppelin] Reamer commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-15 Thread GitBox
Reamer commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1185238379 I understand your motivation. I just don't like the list of Spark configuration at this point, because it can become very long. Spark has very many configuration options and we can't map

[GitHub] [zeppelin] jongyoul commented on pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-15 Thread GitBox
jongyoul commented on PR #4408: URL: https://github.com/apache/zeppelin/pull/4408#issuecomment-1185204305 @zjffdu Let me merge it after finishing CI -- 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] ganlute commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-15 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1185204052 We provide customers with big data cluster services, including HDFS, YARN, Hive, Hbase, Flink, Alluxio, etc. In order to use or manager cluster services more easily, we also provide

[GitHub] [zeppelin] inkkim commented on pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-14 Thread GitBox
inkkim commented on PR #4408: URL: https://github.com/apache/zeppelin/pull/4408#issuecomment-1185182013 @jongyoul Yes, It was resolved again : ) -- 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

[GitHub] [zeppelin] jongyoul commented on pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-14 Thread GitBox
jongyoul commented on PR #4408: URL: https://github.com/apache/zeppelin/pull/4408#issuecomment-1185166120 @inkkim Sorry for bothering you but could you please resolve the conflict again? We are contributing interpreter maintenance issues recently so it can make conflicts again and again.

[GitHub] [zeppelin] jongyoul merged pull request #4405: [ZEPPELIN-5764] Remove Scalding interpreter

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

[GitHub] [zeppelin] jongyoul commented on pull request #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-14 Thread GitBox
jongyoul commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1185163411 @ganlute Hello ~ First of all, thank you for the contribution. BTW, could you please share your experiences with Alluxio in Zeppelin? I like this kind of contribution but we are trying

[GitHub] [zeppelin] ganlute opened a new pull request, #4410: [ZEPPELIN-5771]Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0

2022-07-14 Thread GitBox
ganlute opened a new pull request, #4410: URL: https://github.com/apache/zeppelin/pull/4410 ### What is this PR for? Zeppelin Interpreter Support Alluxio From 1.0.0 To 2.5.0 ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira

[GitHub] [zeppelin] proceane opened a new pull request, #4409: [ZEPPELIN-3945] Create .gitattributes

2022-07-14 Thread GitBox
proceane opened a new pull request, #4409: URL: https://github.com/apache/zeppelin/pull/4409 ### What is this PR for? Create .gitattributes file in root path ### What type of PR is it? Bug Fix ### Todos * [x] - Create .gitattributes file in root path ### What is

[GitHub] [zeppelin] inkkim commented on pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-14 Thread GitBox
inkkim commented on PR #4408: URL: https://github.com/apache/zeppelin/pull/4408#issuecomment-1184304502 @zjffdu Yes, It was resolved. -- 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] zjffdu commented on pull request #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-14 Thread GitBox
zjffdu commented on PR #4408: URL: https://github.com/apache/zeppelin/pull/4408#issuecomment-1184249767 @inkkim Could you resolve the conflicts first? -- 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

[GitHub] [zeppelin] jongyoul merged pull request #4406: [ZEPPELIN-5723] Remove Kylin interpreter

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

[GitHub] [zeppelin] inkkim opened a new pull request, #4408: [ZEPPELIN-5768] Remove scio interpreter

2022-07-13 Thread GitBox
inkkim opened a new pull request, #4408: URL: https://github.com/apache/zeppelin/pull/4408 ### What is this PR for? Removing an interpreter unused by retirement ### What type of PR is it? Refactoring *Please leave your type of PR only* ### Todos * [X] - Remove scio

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-13 Thread GitBox
jongyoul commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r919976353 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message;

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-13 Thread GitBox
Reamer commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r919929896 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message; -import

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4407: [ZEPPELIN-5769] Roles in API-Request

2022-07-13 Thread GitBox
jongyoul commented on code in PR #4407: URL: https://github.com/apache/zeppelin/pull/4407#discussion_r919802360 ## zeppelin-server/src/main/java/org/apache/zeppelin/rest/message/NewNoteRequest.java: ## @@ -16,30 +16,33 @@ */ package org.apache.zeppelin.rest.message;

[GitHub] [zeppelin] Reamer opened a new pull request, #4407: Roles in api

2022-07-13 Thread GitBox
Reamer opened a new pull request, #4407: URL: https://github.com/apache/zeppelin/pull/4407 ### What is this PR for? This pull request adds the roles of a user to API request. See `zeppelin-server/src/main/java/org/apache/zeppelin/rest/AbstractRestApi.java`- Various Rest-API classes

[GitHub] [zeppelin] dntjr8096 commented on a diff in pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop3 profile to true for successful build.

2022-07-12 Thread GitBox
dntjr8096 commented on code in PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#discussion_r919633541 ## .github/workflows/core.yml: ## @@ -448,3 +448,29 @@ jobs: R -e "IRkernel::installspec()" - name: run tests run: ./mvnw verify -DskipRat

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop3 profile to true for successful build.

2022-07-12 Thread GitBox
jongyoul commented on code in PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#discussion_r919600069 ## .github/workflows/core.yml: ## @@ -448,3 +448,29 @@ jobs: R -e "IRkernel::installspec()" - name: run tests run: ./mvnw verify -DskipRat

[GitHub] [zeppelin] dntjr8096 commented on pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop3 profile to true for successful build.

2022-07-12 Thread GitBox
dntjr8096 commented on PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#issuecomment-1181822112 ok. I ran a git-rebase and added a 'default-build' workflow. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [zeppelin] Reamer commented on pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop3 profile to true for successful build.

2022-07-12 Thread GitBox
Reamer commented on PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#issuecomment-1181797768 Please run a git-rebase to the current master. -- 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

[GitHub] [zeppelin] yaini opened a new pull request, #4406: [ZEPPELIN-5723] Remove Kyiln interpreter

2022-07-12 Thread GitBox
yaini opened a new pull request, #4406: URL: https://github.com/apache/zeppelin/pull/4406 ### What is this PR for? Removing an interpreter unused by retirement ### What type of PR is it? Refactoring ### Todos * [x] - Remove Kylin interpreter and related things from the

[GitHub] [zeppelin] SIOUkoeran commented on pull request #4405: [ZEPPELIN-5764] Remove scalding

2022-07-11 Thread GitBox
SIOUkoeran commented on PR #4405: URL: https://github.com/apache/zeppelin/pull/4405#issuecomment-1181276498 > Thank you for doing it. Could you please two more parts which are mentioned below? > >

[GitHub] [zeppelin] SIOUkoeran opened a new pull request, #4405: [ZEPPELIN-5764] Remove scalding

2022-07-11 Thread GitBox
SIOUkoeran opened a new pull request, #4405: URL: https://github.com/apache/zeppelin/pull/4405 ### What is this PR for? Removing Scalding interpreter unused by retirement ### What type of PR is it? [Refactoring] ### Todos * [x] - Remove Scalding interpreter and related

[GitHub] [zeppelin] zjffdu opened a new pull request, #4404: [ZEPPELIN-5761] Support flink 1.15.1

2022-07-11 Thread GitBox
zjffdu opened a new pull request, #4404: URL: https://github.com/apache/zeppelin/pull/4404 ### What is this PR for? Trivial PR to support Flink 1.15.1 ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira issue? *

[GitHub] [zeppelin] zjffdu merged pull request #4388: [ZEPPELIN-5755] Support Spark 3.3

2022-07-11 Thread GitBox
zjffdu merged PR #4388: URL: https://github.com/apache/zeppelin/pull/4388 -- 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:

[GitHub] [zeppelin] zlosim commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-11 Thread GitBox
zlosim commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1180703056 Hi @Reamer , thanks for the clarification. Yes, one can set env vars and create custom interpreter-spec but this is not the same as this fix is trying to do and is not on feature

[GitHub] [zeppelin] Reamer commented on pull request #4400: ZEPPELIN-3527. replace log4j with reload4j

2022-07-11 Thread GitBox
Reamer commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1180424204 > @Reamer , greetings ! i am working on it to completely replace log4j with log4j2 and remove the usage of `slf4j-log4j12` please allow me sometime to fix the error out and raise a pr

[GitHub] [zeppelin] Reamer commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-11 Thread GitBox
Reamer commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1180412197 HI @zlosim As you can see in the following lines, SPARK_SUBMIT_OPTIONS is concatenated with the spark configurations specifically for Kubernetes.

[GitHub] [zeppelin] Samrat002 commented on pull request #4400: ZEPPELIN-3527. replace log4j with reload4j

2022-07-11 Thread GitBox
Samrat002 commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1180371673 @Reamer , greetings ! i am working on it to completely replace log4j with log4j2 and remove the usage of `slf4j-log4j12` please allow me sometime to fix the error out and raise a

[GitHub] [zeppelin] Samrat002 commented on pull request #4400: ZEPPELIN-3527. replace log4j with reload4j

2022-07-11 Thread GitBox
Samrat002 commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1180369633 @jongyoul , greetings!!! I tried changing to log4j2 , and faced some of the issues which need many changes on entire logging As you must be aware that log4j2 has many

[GitHub] [zeppelin] jongyoul merged pull request #4401: Bump moment from 2.29.2 to 2.29.4 in /zeppelin-web

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

[GitHub] [zeppelin] zlosim commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-11 Thread GitBox
zlosim commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1180340269 Hi @Reamer I'm sorry but I am not sure if I understand you correctly. As you stated, spark configs can be included in the `SPARK_SUBMIT_OPTIONS ` but they were NOT. This fix

[GitHub] [zeppelin] Reamer merged pull request #4390: [ZEPPELIN-5757] Assume role in NotebookRepo S3

2022-07-10 Thread GitBox
Reamer merged PR #4390: URL: https://github.com/apache/zeppelin/pull/4390 -- 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:

[GitHub] [zeppelin] Reamer commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-07-10 Thread GitBox
Reamer commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1179771490 I think most Spark configuration values can be included in the environment variable `SPARK_SUBMIT_OPTIONS`. Please note that the K8sProcess works for all Zeppelin interpreters. --

[GitHub] [zeppelin] Reamer commented on pull request #4400: ZEPPELIN-3527. replace log4j with reload4j

2022-07-10 Thread GitBox
Reamer commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1179770489 We should remove `slf4j-log4j12` completely and replace it with the slf4j-api in the zeppelin code. Instead of `log4j1` we should ship `log4j2`. -- This is an automated message from

[GitHub] [zeppelin] Reamer commented on pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop2 profile to true for successful build.

2022-07-10 Thread GitBox
Reamer commented on PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#issuecomment-1179769338 Instead of Hadoop 2, we should prefer Hadoop 3. -- 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

[GitHub] [zeppelin] jongyoul commented on pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop2 profile to true for successful build.

2022-07-10 Thread GitBox
jongyoul commented on PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#issuecomment-1179742276 > It's better to update .github/workflows/core.yml to verify this change. Do you mean to add a new task to confirm the compilation itself? -- This is an automated message from

[GitHub] [zeppelin] zjffdu commented on pull request #4403: [ZEPPELIN-5762] Modify activeByDefault value of hadoop2 profile to true for successful build.

2022-07-10 Thread GitBox
zjffdu commented on PR #4403: URL: https://github.com/apache/zeppelin/pull/4403#issuecomment-1179741691 It's better to update `.github/workflows/core.yml` to verify this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [zeppelin] dntjr8096 closed pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-07-10 Thread GitBox
dntjr8096 closed pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes… URL: https://github.com/apache/zeppelin/pull/4402 -- 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

<    3   4   5   6   7   8   9   10   11   12   >