[jira] [Created] (ZEPPELIN-5762) Resolve hadoop dependency when building zeppelin-server without any profile

2022-07-10 Thread Woosuk Ro (Jira)
Woosuk Ro created ZEPPELIN-5762: --- Summary: Resolve hadoop dependency when building zeppelin-server without any profile Key: ZEPPELIN-5762 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5762 Project

[GitHub] [zeppelin] jongyoul merged pull request #4386: [MINOR] Set permissions for GitHub actions

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

[GitHub] [zeppelin] jongyoul commented on pull request #4386: [MINOR] Set permissions for GitHub actions

2022-07-10 Thread GitBox
jongyoul commented on PR #4386: URL: https://github.com/apache/zeppelin/pull/4386#issuecomment-1179704146 Sorry for the late. Thank you for the contribution. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4401: Bump moment from 2.29.2 to 2.29.4 in /zeppelin-web

2022-07-10 Thread GitBox
dependabot[bot] opened a new pull request, #4401: URL: https://github.com/apache/zeppelin/pull/4401 Bumps [moment](https://github.com/moment/moment) from 2.29.2 to 2.29.4. Changelog Sourced from https://github.com/moment/moment/blob/develop/CHANGELOG.md";>moment's changelog.

[GitHub] [zeppelin] naveensrinivasan commented on pull request #4386: [MINOR] Set permissions for GitHub actions

2022-07-10 Thread GitBox
naveensrinivasan commented on PR #4386: URL: https://github.com/apache/zeppelin/pull/4386#issuecomment-1179706327 > Sorry for the late. Thank you for the contribution. Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

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

2022-07-10 Thread GitBox
dntjr8096 opened a new pull request, #4402: URL: https://github.com/apache/zeppelin/pull/4402 ### What is this PR for? When building from source code, if the Profile option is not given to the maven command by default, an error like Screenshot occurs. So I changed the default profile act

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-07-10 Thread GitBox
jongyoul commented on code in PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#discussion_r917400381 ## zeppelin-server/pom.xml: ## @@ -457,9 +457,6 @@ using-source-tree - Review Comment: This is also needed. Please revert this change. --

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-07-10 Thread GitBox
jongyoul commented on code in PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#discussion_r917400381 ## zeppelin-server/pom.xml: ## @@ -457,9 +457,6 @@ using-source-tree - Review Comment: This is also needed for running integration tests. Pl

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

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

2022-07-10 Thread GitBox
dntjr8096 commented on code in PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#discussion_r917401489 ## zeppelin-server/pom.xml: ## @@ -457,9 +457,6 @@ using-source-tree - Review Comment: Yes. Instead of deleting the line, I changed the act

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-07-10 Thread GitBox
jongyoul commented on code in PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#discussion_r917401933 ## zeppelin-server/pom.xml: ## @@ -457,9 +457,6 @@ using-source-tree - Review Comment: Yes, that should be reasonable but I think it's bette

[GitHub] [zeppelin] dntjr8096 commented on a diff in pull request #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-07-10 Thread GitBox
dntjr8096 commented on code in PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#discussion_r917402330 ## zeppelin-server/pom.xml: ## @@ -457,9 +457,6 @@ using-source-tree - Review Comment: ok i'll do that -- This is an automated message

[GitHub] [zeppelin] dntjr8096 opened a new pull request, #4403: [ZEPPELIN-5762] Change default profile activation to build successfully

2022-07-10 Thread GitBox
dntjr8096 opened a new pull request, #4403: URL: https://github.com/apache/zeppelin/pull/4403 ### What is this PR for? When building from the source code, the default profile activation option of the zeppelin-server module is "using-source-tree", so I get the error as in the screenshot.

[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 t

[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

[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 t

[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 g

[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 #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. -- This

[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: dev-unsubscr...@zeppelin.apach

[jira] [Created] (ZEPPELIN-5763) when I run spark sql zeppelin throw "java.lang.NoClassDefFoundError: org/apache/commons/lang/StringUtils"

2022-07-10 Thread Hongwei Lu (Jira)
Hongwei Lu created ZEPPELIN-5763: Summary: when I run spark sql zeppelin throw "java.lang.NoClassDefFoundError: org/apache/commons/lang/StringUtils" Key: ZEPPELIN-5763 URL: https://issues.apache.org/jira/browse/ZE