[GitHub] [zeppelin] jongyoul merged pull request #4493: Bump commons-text from 1.8 to 1.10.0

2022-10-25 Thread GitBox
jongyoul merged PR #4493: URL: https://github.com/apache/zeppelin/pull/4493 -- 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] Reamer commented on pull request #4497: [ZEPPELIN-5207] Build with JDK 11 in CI

2022-10-25 Thread GitBox
Reamer commented on PR #4497: URL: https://github.com/apache/zeppelin/pull/4497#issuecomment-1290339375 In my opinion this pretty much overloads the CI system, we should go for one JDK version for most of the testing. The default-build job could be done with "all" JDKs. How do you guys see

[GitHub] [zeppelin] Reamer opened a new pull request, #4497: [ZEPPELIN-5207] Build with JDK 11 in CI and Docker

2022-10-25 Thread GitBox
Reamer opened a new pull request, #4497: URL: https://github.com/apache/zeppelin/pull/4497 ### What is this PR for? This PR adds JDK 11 in the CI. ### What type of PR is it? Improvement ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-520

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4496: Bump flink.hadoop.version from 2.7.7 to 3.1.1.7.2.15.0-147 in /flink/flink-scala-parent

2022-10-24 Thread GitBox
dependabot[bot] opened a new pull request, #4496: URL: https://github.com/apache/zeppelin/pull/4496 Bumps `flink.hadoop.version` from 2.7.7 to 3.1.1.7.2.15.0-147. Updates `hadoop-common` from 2.7.7 to 3.1.1.7.2.15.0-147 Updates `hadoop-hdfs` from 2.7.7 to 3.1.1.7.2.15.0-147 U

[GitHub] [zeppelin] dependabot[bot] closed pull request #4352: Bump hadoop-common from 2.6.5 to 3.2.3 in /flink/flink-scala-parent

2022-10-24 Thread GitBox
dependabot[bot] closed pull request #4352: Bump hadoop-common from 2.6.5 to 3.2.3 in /flink/flink-scala-parent URL: https://github.com/apache/zeppelin/pull/4352 -- 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] commented on pull request #4352: Bump hadoop-common from 2.6.5 to 3.2.3 in /flink/flink-scala-parent

2022-10-24 Thread GitBox
dependabot[bot] commented on PR #4352: URL: https://github.com/apache/zeppelin/pull/4352#issuecomment-1290057348 Looks like org.apache.hadoop:hadoop-common is updatable in another way, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] Reamer merged pull request #4494: [ZEPPELIN-5836] Remove scala-2.10 because it has no JDK 11 Support

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

[GitHub] [zeppelin] zjffdu commented on pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-10-24 Thread GitBox
zjffdu commented on PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#issuecomment-1289875755 LGTM, 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 spe

[GitHub] [zeppelin] Reamer commented on pull request #4494: [ZEPPELIN-5836] Remove scala-2.10 because it has no JDK 11 Support

2022-10-24 Thread GitBox
Reamer commented on PR #4494: URL: https://github.com/apache/zeppelin/pull/4494#issuecomment-1288769099 @jongyoul Now the core tests are running. I would be happy about a review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [zeppelin] Reamer merged pull request #4439: [ZEPPELIN-5191] Fix the wrong Docker image name in zeppelin-server.yaml

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

[GitHub] [zeppelin] asfgit closed pull request #4495: [ZEPPELIN-5819] Configures ivy to download jar libraries only from remote

2022-10-23 Thread GitBox
asfgit closed pull request #4495: [ZEPPELIN-5819] Configures ivy to download jar libraries only from remote URL: https://github.com/apache/zeppelin/pull/4495 -- 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] Reamer commented on pull request #4494: [ZEPPELIN-5836] Remove scala-2.10 because it has no JDK 11 Support

2022-10-21 Thread GitBox
Reamer commented on PR #4494: URL: https://github.com/apache/zeppelin/pull/4494#issuecomment-1286986545 Some tests are not yet running correctly. I think the Ivy change #4495 should bring more stability. -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [zeppelin] Reamer opened a new pull request, #4495: [ZEPPELIN-5819] Configures ivy to download jar libraries only from remote

2022-10-21 Thread GitBox
Reamer opened a new pull request, #4495: URL: https://github.com/apache/zeppelin/pull/4495 ### What is this PR for? Configures ivy to download jar libraries only from remote and ignore local jars ### What type of PR is it? * Hot Fix ### What is the Jira issue? * http

[GitHub] [zeppelin] jongyoul commented on pull request #4494: [ZEPPELIN-5836] Remove scala-2.10 because it has no JDK 11 Support

2022-10-20 Thread GitBox
jongyoul commented on PR #4494: URL: https://github.com/apache/zeppelin/pull/4494#issuecomment-1285465831 Oh! It's still draft but I think it's really necessary job. Thank you -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [zeppelin] Reamer opened a new pull request, #4494: [ZEPPELIN-5836] Remove scala-2.10 because it has no JDK 11 Support

2022-10-20 Thread GitBox
Reamer opened a new pull request, #4494: URL: https://github.com/apache/zeppelin/pull/4494 ### What is this PR for? This PR removes scala 2.10 because of missing support for JDK 11. It also changes the scala compile maven plugin to `net.alchim31.maven`. ### What type of PR is it?

[GitHub] [zeppelin] jongyoul commented on pull request #4479: [ZEPPELIN-5833] Enable Cassandra for JDK11

2022-10-20 Thread GitBox
jongyoul commented on PR #4479: URL: https://github.com/apache/zeppelin/pull/4479#issuecomment-1285442430 FYI, I heard that @alexott is not actively contributing to Cassandra any longer. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4493: Bump commons-text from 1.8 to 1.10.0

2022-10-20 Thread GitBox
dependabot[bot] opened a new pull request, #4493: URL: https://github.com/apache/zeppelin/pull/4493 Bumps commons-text from 1.8 to 1.10.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.commons:commo

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4492: Bump commons-text from 1.8 to 1.10.0 in /zeppelin-client

2022-10-20 Thread GitBox
dependabot[bot] opened a new pull request, #4492: URL: https://github.com/apache/zeppelin/pull/4492 Bumps commons-text from 1.8 to 1.10.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.commons:commo

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4491: Bump hadoop.version from 2.7.7 to 3.1.1.7.2.15.0-147 in /livy

2022-10-20 Thread GitBox
dependabot[bot] opened a new pull request, #4491: URL: https://github.com/apache/zeppelin/pull/4491 Bumps `hadoop.version` from 2.7.7 to 3.1.1.7.2.15.0-147. Updates `hadoop-auth` from 2.7.7 to 3.1.1.7.2.15.0-147 Updates `hadoop-common` from 2.7.7 to 3.1.1.7.2.15.0-147 Updates

[GitHub] [zeppelin] dependabot[bot] closed pull request #4349: Bump hadoop-common from 2.6.5 to 3.2.3 in /livy

2022-10-20 Thread GitBox
dependabot[bot] closed pull request #4349: Bump hadoop-common from 2.6.5 to 3.2.3 in /livy URL: https://github.com/apache/zeppelin/pull/4349 -- 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 speci

[GitHub] [zeppelin] dependabot[bot] commented on pull request #4349: Bump hadoop-common from 2.6.5 to 3.2.3 in /livy

2022-10-20 Thread GitBox
dependabot[bot] commented on PR #4349: URL: https://github.com/apache/zeppelin/pull/4349#issuecomment-1285358225 Looks like org.apache.hadoop:hadoop-common is updatable in another way, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] asfgit closed pull request #4484: [ZEPPELIN-5834] Remove last references to Hadoop 2.6 as JDK11 is not supported.

2022-10-20 Thread GitBox
asfgit closed pull request #4484: [ZEPPELIN-5834] Remove last references to Hadoop 2.6 as JDK11 is not supported. URL: https://github.com/apache/zeppelin/pull/4484 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [zeppelin] asfgit closed pull request #4479: [ZEPPELIN-5833] Enable Cassandra for JDK11

2022-10-20 Thread GitBox
asfgit closed pull request #4479: [ZEPPELIN-5833] Enable Cassandra for JDK11 URL: https://github.com/apache/zeppelin/pull/4479 -- 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. T

[GitHub] [zeppelin] zjffdu merged pull request #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

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

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4490: Bump jettison from 1.4.0 to 1.5.1

2022-10-18 Thread GitBox
dependabot[bot] opened a new pull request, #4490: URL: https://github.com/apache/zeppelin/pull/4490 Bumps [jettison](https://github.com/jettison-json/jettison) from 1.4.0 to 1.5.1. Release notes Sourced from https://github.com/jettison-json/jettison/releases";>jettison's releases.

[GitHub] [zeppelin] Reamer commented on pull request #4484: [ZEPPELIN-5834] Remove last references to Hadoop 2.6 as JDK11 is not supported.

2022-10-18 Thread GitBox
Reamer commented on PR #4484: URL: https://github.com/apache/zeppelin/pull/4484#issuecomment-1281905081 If no further comments are received till Thursday, I will merge the change into the master to proceed quickly with JDK 11 support. -- This is an automated message from the Apache Git Se

[GitHub] [zeppelin] Reamer commented on pull request #4479: [ZEPPELIN-5833] Enable Cassandra for JDK11

2022-10-18 Thread GitBox
Reamer commented on PR #4479: URL: https://github.com/apache/zeppelin/pull/4479#issuecomment-1281903978 If no further comments are received till Thursday, I will merge the change into the master to proceed quickly with JDK 11 support. -- This is an automated message from the Apache Git Se

[GitHub] [zeppelin] Reamer commented on pull request #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

2022-10-14 Thread GitBox
Reamer commented on PR #4478: URL: https://github.com/apache/zeppelin/pull/4478#issuecomment-1278621303 @zjffdu I am not familiar with the homepage. Please merge the PullRequest if everything fits. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] zkytech commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-13 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1278502576 @zjffdu To fix [this bug](https://github.com/apache/zeppelin/pull/4443#issuecomment-1214710538), I have made a change in [this place](https://github.com/apache/zeppelin/pull/4443/files?

[GitHub] [zeppelin] zkytech commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-13 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1278484075 [Unit test](https://github.com/apache/zeppelin/blob/master/livy/src/test/java/org/apache/zeppelin/livy/LivySQLInterpreterTest.java#L193) is required to update. Because JSON number is no

[GitHub] [zeppelin] zjffdu commented on pull request #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

2022-10-13 Thread GitBox
zjffdu commented on PR #4478: URL: https://github.com/apache/zeppelin/pull/4478#issuecomment-1278390838 Thanks @raboof , PR 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 comme

[GitHub] [zeppelin] Reamer opened a new pull request, #4484: [ZEPPELIN-5834] Remove last references to Hadoop 2.6 as JDK11 is not supported.

2022-10-13 Thread GitBox
Reamer opened a new pull request, #4484: URL: https://github.com/apache/zeppelin/pull/4484 ### What is this PR for? This PR updates Hadoop 2.6 dependencies to 2.7. ### What type of PR is it? Improvement ### What is the Jira issue? * https://issues.apache.org/jira/browse

[GitHub] [zeppelin] Reamer commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-13 Thread GitBox
Reamer commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1277544458 > @Reamer Done Let's wait for CI tests -- 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] zkytech commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-13 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1277541818 @Reamer Done -- 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 uns

[GitHub] [zeppelin] dependabot[bot] closed pull request #4371: Bump gson from 2.8.6 to 2.8.9

2022-10-13 Thread GitBox
dependabot[bot] closed pull request #4371: Bump gson from 2.8.6 to 2.8.9 URL: https://github.com/apache/zeppelin/pull/4371 -- 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 un

[GitHub] [zeppelin] dependabot[bot] commented on pull request #4371: Bump gson from 2.8.6 to 2.8.9

2022-10-13 Thread GitBox
dependabot[bot] commented on PR #4371: URL: https://github.com/apache/zeppelin/pull/4371#issuecomment-1277510633 Looks like com.google.code.gson:gson 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, ple

[GitHub] [zeppelin] Reamer merged pull request #4476: Upgrade gson to 2.8.9 due to CVE-2022-25647

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

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4481: Bump shiro-core from 1.9.1 to 1.10.0

2022-10-12 Thread GitBox
dependabot[bot] opened a new pull request, #4481: URL: https://github.com/apache/zeppelin/pull/4481 Bumps [shiro-core](https://github.com/apache/shiro) from 1.9.1 to 1.10.0. Commits https://github.com/apache/shiro/commit/7141f41137f058a3c756ea13ed4e0fef9a3abb66";>7141f41 [maven

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

2022-10-12 Thread GitBox
Reamer commented on code in PR #4478: URL: https://github.com/apache/zeppelin/pull/4478#discussion_r993577573 ## security.md: ## @@ -0,0 +1,75 @@ +# Zeppelin Security + +This page explains what security characteristics can be expected from +Zeppelin, what measures operators of a

[GitHub] [zeppelin] Reamer commented on pull request #4441: [ZEPPELIN-4871] Fix livy interpreter 0.7.1 donwload link

2022-10-12 Thread GitBox
Reamer commented on PR #4441: URL: https://github.com/apache/zeppelin/pull/4441#issuecomment-1276272601 Please rebase your PR to 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 go to th

[GitHub] [zeppelin] Reamer commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-12 Thread GitBox
Reamer commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1276272264 Please rebase your PR to 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 go to th

[GitHub] [zeppelin] Reamer commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-12 Thread GitBox
Reamer commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1276272178 Please rebase your PR to 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 go to th

[GitHub] [zeppelin] Reamer merged pull request #4470: update livy archive name

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

[GitHub] [zeppelin] Reamer opened a new pull request, #4480: [ZEPPELIN-5832] Use protoc-jar-maven-plugin, which has a good IDE support

2022-10-12 Thread GitBox
Reamer opened a new pull request, #4480: URL: https://github.com/apache/zeppelin/pull/4480 ### What is this PR for? This PR changes the plugin to compile the proto files. The Eclipse IDE support is very good and it works immediately after integration. ### What type of PR is it?

[GitHub] [zeppelin] Reamer commented on pull request #4479: [ZEPPELIN-5831] Enable Cassandra for JDK11

2022-10-12 Thread GitBox
Reamer commented on PR #4479: URL: https://github.com/apache/zeppelin/pull/4479#issuecomment-1276135889 I have not tested the changes against any external Cassandra backend. Besides, my Cassandra knowledge is very limited. I would be very grateful if someone tests the changes. Maybe @alex

[GitHub] [zeppelin] Reamer opened a new pull request, #4479: [ZEPPELIN-5831] Enable Cassandra for JDK11

2022-10-12 Thread GitBox
Reamer opened a new pull request, #4479: URL: https://github.com/apache/zeppelin/pull/4479 ### What is this PR for? This PR updates scala for the cassandra interpreter to 2.12 ### What type of PR is it? - Improvement ### Todos * [ ] - Verify with real Cassandra Back

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

2022-10-12 Thread GitBox
Reamer commented on code in PR #4478: URL: https://github.com/apache/zeppelin/pull/4478#discussion_r993352854 ## security.md: ## @@ -0,0 +1,75 @@ +# Zeppelin Security + +This page explains what security characteristics can be expected from +Zeppelin, what measures operators of a

[GitHub] [zeppelin] raboof opened a new pull request, #4478: [ZEPPELIN-5830] Add Zeppelin Security page to the website

2022-10-12 Thread GitBox
raboof opened a new pull request, #4478: URL: https://github.com/apache/zeppelin/pull/4478 ### What is this PR for? This PR adds a 'Security' page to the website explaining what security characteristics people can expect from Zeppelin, and how to report any security issues people mig

[GitHub] [zeppelin] Reamer commented on pull request #4477: fix(sec): upgrade org.codehaus.groovy:groovy-all to 2.4.8

2022-10-12 Thread GitBox
Reamer commented on PR #4477: URL: https://github.com/apache/zeppelin/pull/4477#issuecomment-1275750436 Hi @Super-Sky Thank you very much for your contribution. Please take out the file formatting from your commit. -- This is an automated message from the Apache Git Service. To respon

[GitHub] [zeppelin] zkytech commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-12 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1275739570 @zjffdu fixed -- 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 un

[GitHub] [zeppelin] PrimWILL closed pull request #4465: [ZEPPELIN-5206] JDK 11 Support

2022-10-11 Thread GitBox
PrimWILL closed pull request #4465: [ZEPPELIN-5206] JDK 11 Support URL: https://github.com/apache/zeppelin/pull/4465 -- 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 unsubscr

[GitHub] [zeppelin] PrimWILL commented on pull request #4465: [ZEPPELIN-5206] JDK 11 Support

2022-10-11 Thread GitBox
PrimWILL commented on PR #4465: URL: https://github.com/apache/zeppelin/pull/4465#issuecomment-1274889389 > In the next few days, I'll try to split the changes into several pull requests. That's great news! Then I'll close this draft pr and check the PR that you'll make =) -- This

[GitHub] [zeppelin] Reamer commented on pull request #4465: [ZEPPELIN-5206] JDK 11 Support

2022-10-11 Thread GitBox
Reamer commented on PR #4465: URL: https://github.com/apache/zeppelin/pull/4465#issuecomment-1274855825 In the last few days I've been very active working on the [branch](https://github.com/Reamer/zeppelin/tree/jdk11), as the connection libraries for Spark with JDK-8 are slowly breaking awa

[GitHub] [zeppelin] Reamer commented on pull request #4459: [ZEPPELIN-5815] Frontend CI jobs improvement

2022-10-11 Thread GitBox
Reamer commented on PR #4459: URL: https://github.com/apache/zeppelin/pull/4459#issuecomment-1274532079 Sorry if I was vague here. Let me describe it in a little more detail. During the tests the project `zeppelin-integration` is executed. In my opinion this sub-module has to define all d

[GitHub] [zeppelin] zkytech commented on pull request #4423: [ZEPPELIN-5781]Add cross datasource query support to Spark SQL interpreter

2022-10-11 Thread GitBox
zkytech commented on PR #4423: URL: https://github.com/apache/zeppelin/pull/4423#issuecomment-1274275116 I have not found a suitable way to get interpreter settings nor make access control work as expect. You can close this PR, If I have any progress on this, I will create a new PR.

[GitHub] [zeppelin] huage1994 opened a new pull request, #4476: Upgrade gson to 2.8.9 due to CVE-2022-25647

2022-10-10 Thread GitBox
huage1994 opened a new pull request, #4476: URL: https://github.com/apache/zeppelin/pull/4476 ### What is this PR for? Upgrade gson to 2.8.9 due to CVE-2022-25647 ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira issue? * O

[GitHub] [zeppelin] huage1994 commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-10-10 Thread GitBox
huage1994 commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1274025896 Hi @jongyoul @zjffdu @Reamer . I tried your suggestion but the problem still exists. **I think there are some dependencies in `launcher-k8s-standard` conflict with junit 5**

[GitHub] [zeppelin] zjffdu commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-10-10 Thread GitBox
zjffdu commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1273430954 @zkytech livy interpreter in CI is failed, could help fix it? [LivySparkSQLInterpreter](https://github.com/apache/zeppelin/actions/runs/3117336852/jobs/5055926704) -- This is an automat

[GitHub] [zeppelin] maziyarpanahi commented on pull request #4439: [ZEPPELIN-5191] Fix the wrong Docker image name in zeppelin-server.yaml

2022-10-10 Thread GitBox
maziyarpanahi commented on PR #4439: URL: https://github.com/apache/zeppelin/pull/4439#issuecomment-1273335199 > Very good, that is how I imagined it. > > I would remove the change in `k8s/zeppelin-server.yaml` and put a reminder in the documentation to adjust the images in the YAML-F

[GitHub] [zeppelin] Reamer commented on pull request #4439: [ZEPPELIN-5191] Fix the wrong Docker image name in zeppelin-server.yaml

2022-10-10 Thread GitBox
Reamer commented on PR #4439: URL: https://github.com/apache/zeppelin/pull/4439#issuecomment-1273246694 Very good, that is how I imagined it. I would remove the change in `k8s/zeppelin-server.yaml` and put a reminder in the documentation to adjust the images in the YAML-File. -- Th

[GitHub] [zeppelin] maziyarpanahi commented on pull request #4439: [ZEPPELIN-5191] Fix the wrong Docker image name in zeppelin-server.yaml

2022-10-10 Thread GitBox
maziyarpanahi commented on PR #4439: URL: https://github.com/apache/zeppelin/pull/4439#issuecomment-1273182235 Hi @Reamer I have added the steps to build `zeppelin-distribution`, `zeppelin-server`, and `zeppelin-interpreter` to be set inside `./k8s/zeppelin-server.yaml` script.

[GitHub] [zeppelin] huage1994 commented on pull request #4459: [ZEPPELIN-5815] Frontend CI jobs improvement

2022-10-10 Thread GitBox
huage1994 commented on PR #4459: URL: https://github.com/apache/zeppelin/pull/4459#issuecomment-1272967709 > Zeppelin uses maven 3.8.1 . mvn -pl parent-module would ignores any child modules. `/mvnw clean package -pl zeppelin-plugins -amd -B` `-amd` is a trick to solve this proble

[GitHub] [zeppelin] huage1994 commented on pull request #4459: [ZEPPELIN-5815] Frontend CI jobs improvement

2022-10-10 Thread GitBox
huage1994 commented on PR #4459: URL: https://github.com/apache/zeppelin/pull/4459#issuecomment-1272962124 > In general, I am against additional Maven calls. Maybe it makes sense to work with the Maven parameters "-am" and "-pl" in the first call. This should build only the most necessary a

[GitHub] [zeppelin] Reamer commented on pull request #4459: [ZEPPELIN-5815] Frontend CI jobs improvement

2022-10-08 Thread GitBox
Reamer commented on PR #4459: URL: https://github.com/apache/zeppelin/pull/4459#issuecomment-1272467375 In general, I am against additional Maven calls. Maybe it makes sense to work with the Maven parameters "-am" and "-pl" in the first call. This should build only the most necessary arte

[GitHub] [zeppelin] zjffdu commented on pull request #4459: [ZEPPELIN-5815] Frontend CI jobs improvement

2022-10-06 Thread GitBox
zjffdu commented on PR #4459: URL: https://github.com/apache/zeppelin/pull/4459#issuecomment-1270992699 Frontend CI is passed,Thanks @huage1994 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 t

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4475: Bump css-what from 2.1.2 to 2.1.3 in /zeppelin-web

2022-10-05 Thread GitBox
dependabot[bot] opened a new pull request, #4475: URL: https://github.com/apache/zeppelin/pull/4475 Bumps [css-what](https://github.com/fb55/css-what) from 2.1.2 to 2.1.3. Commits https://github.com/fb55/css-what/commit/2db00ca221922c5b5131d798614aa043f2f6f80e";>2db00ca 2.1.3

[GitHub] [zeppelin] huage1994 commented on a diff in pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-10-02 Thread GitBox
huage1994 commented on code in PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#discussion_r985214869 ## jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: ## @@ -550,9 +549,9 @@ public Connection getConnection(String dbPrefix, InterpreterContext conte

[GitHub] [zeppelin] huage1994 commented on a diff in pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-10-02 Thread GitBox
huage1994 commented on code in PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#discussion_r985214869 ## jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: ## @@ -550,9 +549,9 @@ public Connection getConnection(String dbPrefix, InterpreterContext conte

[GitHub] [zeppelin] huage1994 commented on a diff in pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-10-02 Thread GitBox
huage1994 commented on code in PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#discussion_r985214869 ## jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: ## @@ -550,9 +549,9 @@ public Connection getConnection(String dbPrefix, InterpreterContext conte

[GitHub] [zeppelin] huage1994 commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-30 Thread GitBox
huage1994 commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1264239533 > @huage1994 My checkstyle pull request #4466 was merged with the master. Can you please check if you still get the error when you run `mvn install`. Thanks @Reamer ! I'll try t

[GitHub] [zeppelin] Reamer commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-30 Thread GitBox
Reamer commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1263430960 @huage1994 My checkstyle pull request https://github.com/apache/zeppelin/pull/4466 was merged with the master. Can you please check if you still get the error when you run `mvn instal

[GitHub] [zeppelin] jongyoul merged pull request #4466: [ZEPPELIN-5824] Checkstyle

2022-09-30 Thread GitBox
jongyoul merged PR #4466: URL: https://github.com/apache/zeppelin/pull/4466 -- 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 #4466: [ZEPPELIN-5824] Checkstyle

2022-09-30 Thread GitBox
jongyoul commented on PR #4466: URL: https://github.com/apache/zeppelin/pull/4466#issuecomment-1263375267 Me neither to think of a better name. Let's merge it and find it later. Thank you for taking care of it. -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [zeppelin] Reamer merged pull request #4469: Update protobuf-maven-plugin from 0.5.0 to 0.6.1

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

[GitHub] [zeppelin] raboof commented on pull request #4469: Update protobuf-maven-plugin from 0.5.0 to 0.6.1

2022-09-30 Thread GitBox
raboof commented on PR #4469: URL: https://github.com/apache/zeppelin/pull/4469#issuecomment-1263346653 > I saw your issue here. [xolstice/protobuf-maven-plugin#108](https://github.com/xolstice/protobuf-maven-plugin/issues/108) > > Does the update solve your problem? Yes - it s

[GitHub] [zeppelin] Reamer commented on pull request #4469: Update protobuf-maven-plugin from 0.5.0 to 0.6.1

2022-09-30 Thread GitBox
Reamer commented on PR #4469: URL: https://github.com/apache/zeppelin/pull/4469#issuecomment-1263331239 I saw your issue here. https://github.com/xolstice/protobuf-maven-plugin/issues/108 Does the update solve your problem? -- This is an automated message from the Apache Git Serv

[GitHub] [zeppelin] Reamer merged pull request #4468: [ZEPPELIN-5826] Activate credentialReplacement test

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

[GitHub] [zeppelin] Reamer commented on pull request #4466: [ZEPPELIN-5824] Checkstyle

2022-09-29 Thread GitBox
Reamer commented on PR #4466: URL: https://github.com/apache/zeppelin/pull/4466#issuecomment-1262264493 The name of the module comes from the [Checkstyle documentation](https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html). I have found other projects

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-09-29 Thread GitBox
jongyoul commented on code in PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#discussion_r983479720 ## jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: ## @@ -550,9 +549,9 @@ public Connection getConnection(String dbPrefix, InterpreterContext contex

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-09-29 Thread GitBox
jongyoul commented on code in PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#discussion_r983477191 ## jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java: ## @@ -550,9 +549,9 @@ public Connection getConnection(String dbPrefix, InterpreterContext contex

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

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

[GitHub] [zeppelin] huage1994 commented on pull request #4463: [ZEPPELIN-5493] Remove prefix in jdbc interpreter

2022-09-28 Thread GitBox
huage1994 commented on PR #4463: URL: https://github.com/apache/zeppelin/pull/4463#issuecomment-1260759390 > > BTW, don't it have a doc as well? > > Thanks @jongyoul for reminding! I found the [hive related doc](https://zeppelin.apache.org/docs/0.10.1/interpreter/jdbc.html#apache-hive

[GitHub] [zeppelin] raboof closed pull request #4470: update livy archive name

2022-09-27 Thread GitBox
raboof closed pull request #4470: update livy archive name URL: https://github.com/apache/zeppelin/pull/4470 -- 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-m

[GitHub] [zeppelin] raboof commented on pull request #4470: update livy archive name

2022-09-27 Thread GitBox
raboof commented on PR #4470: URL: https://github.com/apache/zeppelin/pull/4470#issuecomment-1259525054 > Please also take a look at #4441 sorry, totally missed that, that is indeed more complete :+1: -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [zeppelin] Reamer commented on pull request #4470: update livy archive name

2022-09-27 Thread GitBox
Reamer commented on PR #4470: URL: https://github.com/apache/zeppelin/pull/4470#issuecomment-1259517372 Please also take a look at #4441 -- 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 specif

[GitHub] [zeppelin] raboof opened a new pull request, #4470: update livy archive name

2022-09-27 Thread GitBox
raboof opened a new pull request, #4470: URL: https://github.com/apache/zeppelin/pull/4470 ### What is this PR for? Fix the download script so it can correctly download with livy 0.6.0 and later ### What type of PR is it? Bug Fix ### How should this be tested?

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

2022-09-27 Thread GitBox
zlosim commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1259198464 Hi @Reamer no prob, should be rebased -- 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 t

[GitHub] [zeppelin] zkytech commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-09-23 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1256880771 > Sorry for late response @zkytech Could you retrigger the CI? ok -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [zeppelin] zjffdu commented on pull request #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-09-23 Thread GitBox
zjffdu commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1256844167 Sorry for late response @zkytech Could you retrigger the CI? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [zeppelin] zjffdu commented on pull request #4441: [ZEPPELIN-4871] Fix livy interpreter 0.7.1 donwload link

2022-09-23 Thread GitBox
zjffdu commented on PR #4441: URL: https://github.com/apache/zeppelin/pull/4441#issuecomment-1256844041 @soicem Any update? -- 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] zjffdu merged pull request #4392: [ZEPPELIN-5694] Support apache kyuubi

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

[GitHub] [zeppelin] raboof opened a new pull request, #4469: Update protobuf-maven-plugin from 0.5.0 to 0.6.1

2022-09-23 Thread GitBox
raboof opened a new pull request, #4469: URL: https://github.com/apache/zeppelin/pull/4469 ### What is this PR for? Update the protobuf plugin to a version that, among other things, no longer overwrites the protoc compiler executable each time the project is built, which is convenien

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

2022-09-23 Thread GitBox
Reamer commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1256315800 Hi @zlosim Sorry for the late reply. I tried to test your branch. Currently, I am getting a startup error because I have already fully transitioned my environment to Spark-3.3. Could

[GitHub] [zeppelin] Reamer opened a new pull request, #4468: [ZEPPELIN-5826] Activate credentialReplacement test

2022-09-22 Thread GitBox
Reamer opened a new pull request, #4468: URL: https://github.com/apache/zeppelin/pull/4468 ### What is this PR for? This pull request activates and corrects the test ParagraphTestcredentialReplacement() ### What type of PR is it? Improvement ### What is the Jira issue?

[GitHub] [zeppelin] Reamer merged pull request #4467: GitHub Workflows security hardening

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

[GitHub] [zeppelin] sashashura opened a new pull request, #4467: GitHub Workflows security hardening

2022-09-20 Thread GitBox
sashashura opened a new pull request, #4467: URL: https://github.com/apache/zeppelin/pull/4467 This PR adds explicit [permissions section](https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions) to workflows. This is a security best practice becaus

[GitHub] [zeppelin] jongyoul commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-20 Thread GitBox
jongyoul commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1252030086 Thank you. I'm checking the PR now -- 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 sp

[GitHub] [zeppelin] Reamer commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-20 Thread GitBox
Reamer commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1252029039 > > I suspect the checkstyle plugin. This plugin produces a lot of the Maven warnings at the beginning. > > I've been wanting to fix this for a long time, but haven't gotten around to

[GitHub] [zeppelin] Reamer opened a new pull request, #4466: [ZEPPELIN-5824] Checkstyle

2022-09-20 Thread GitBox
Reamer opened a new pull request, #4466: URL: https://github.com/apache/zeppelin/pull/4466 ### What is this PR for? This PullRequest introduces the new sub-module "build-tools". This allows the complete Zeppelin project to use the packed Checkstyke configuration centrally. It is not p

[GitHub] [zeppelin] huage1994 commented on pull request #4321: [ZEPPELIN-5612] Upgrade junit 4 to 5

2022-09-19 Thread GitBox
huage1994 commented on PR #4321: URL: https://github.com/apache/zeppelin/pull/4321#issuecomment-1251100264 @zjffdu @Reamer @jongyoul , really thanks for helping out~ I'm going to test your guesses and continue to address this issue -- This is an automated message from the Apache G

<    1   2   3   4   5   6   7   8   9   10   >