[GitHub] [zeppelin] jongyoul commented on pull request #2900: Alluxio

2022-08-25 Thread GitBox
jongyoul commented on PR #2900: URL: https://github.com/apache/zeppelin/pull/2900#issuecomment-1226967945 We are currently providing Alluxio 2.5 so I'll close this PR. Thank you for your interest. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] jongyoul commented on pull request #3507: Bump ffi from 1.9.3 to 1.11.1 in /docs

2022-08-25 Thread GitBox
jongyoul commented on PR #3507: URL: https://github.com/apache/zeppelin/pull/3507#issuecomment-1226965457 @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

[GitHub] [zeppelin] dependabot[bot] closed pull request #4383: Bump elasticsearch from 2.4.3 to 6.8.17 in /elasticsearch

2022-08-25 Thread GitBox
dependabot[bot] closed pull request #4383: Bump elasticsearch from 2.4.3 to 6.8.17 in /elasticsearch URL: https://github.com/apache/zeppelin/pull/4383 -- 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] jongyoul commented on pull request #4383: Bump elasticsearch from 2.4.3 to 6.8.17 in /elasticsearch

2022-08-25 Thread GitBox
jongyoul commented on PR #4383: URL: https://github.com/apache/zeppelin/pull/4383#issuecomment-1226964389 @dependabot close -- 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 #4371: Bump gson from 2.8.6 to 2.8.9

2022-08-25 Thread GitBox
jongyoul commented on PR #4371: URL: https://github.com/apache/zeppelin/pull/4371#issuecomment-1226962239 @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

[GitHub] [zeppelin] jongyoul merged pull request #4380: Bump gson from 2.8.5 to 2.8.9 in /influxdb

2022-08-25 Thread GitBox
jongyoul merged PR #4380: URL: https://github.com/apache/zeppelin/pull/4380 -- 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 #4370: Bump gson from 2.8.2 to 2.8.9 in /zeppelin-integration

2022-08-25 Thread GitBox
jongyoul merged PR #4370: URL: https://github.com/apache/zeppelin/pull/4370 -- 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 #4448: [ZEPPELIN-5802][MINOR] fix rimraf version

2022-08-25 Thread GitBox
jongyoul merged PR #4448: URL: https://github.com/apache/zeppelin/pull/4448 -- 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 #4452: [MINOR] Add some GitHub features for PR

2022-08-25 Thread GitBox
jongyoul merged PR #4452: URL: https://github.com/apache/zeppelin/pull/4452 -- 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 #4452: [MINOR] Add some GitHub features for PR

2022-08-24 Thread GitBox
jongyoul commented on PR #4452: URL: https://github.com/apache/zeppelin/pull/4452#issuecomment-1226804126 @zjffdu Added. -- 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] huage1994 commented on pull request #4392: [ZEPPELIN-5694] Support apache kyuubi

2022-08-24 Thread GitBox
huage1994 commented on PR #4392: URL: https://github.com/apache/zeppelin/pull/4392#issuecomment-1226704414 > Thanks for the contribution @huage1994, is kerbose also supported? And it is better to give example how to enable kyuubi features via jdbc interpreter Hi @zjffdu , kerberos is

[GitHub] [zeppelin] zjffdu commented on pull request #4452: [MINOR] Add some GitHub features for PR

2022-08-24 Thread GitBox
zjffdu commented on PR #4452: URL: https://github.com/apache/zeppelin/pull/4452#issuecomment-1226665856 LGTM @jongyoul Could you update the description? -- 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] dependabot[bot] closed pull request #4393: Bump hadoop-client from 2.6.5 to 2.7.0 in /livy

2022-08-24 Thread GitBox
dependabot[bot] closed pull request #4393: Bump hadoop-client from 2.6.5 to 2.7.0 in /livy URL: https://github.com/apache/zeppelin/pull/4393 -- 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] jongyoul commented on pull request #4393: Bump hadoop-client from 2.6.5 to 2.7.0 in /livy

2022-08-24 Thread GitBox
jongyoul commented on PR #4393: URL: https://github.com/apache/zeppelin/pull/4393#issuecomment-1226656354 @dependabot close -- 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 opened a new pull request, #4452: [MINOR] Add some GitHub features for PR

2022-08-24 Thread GitBox
jongyoul opened a new pull request, #4452: URL: https://github.com/apache/zeppelin/pull/4452 ### 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] kkoldduck2 commented on pull request #4448: [ZEPPELIN-5802][MINOR] fix rimraf version

2022-08-24 Thread GitBox
kkoldduck2 commented on PR #4448: URL: https://github.com/apache/zeppelin/pull/4448#issuecomment-1226620793 > There was also a change in the `zeppelin-web/package-lock.json`. This should also be changed back. Related PR: #4437 Oh! I will change it back too! thank you -- This is

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4451: Bump engine.io and karma in /zeppelin-web-angular

2022-08-24 Thread GitBox
dependabot[bot] opened a new pull request, #4451: URL: https://github.com/apache/zeppelin/pull/4451 Bumps [engine.io](https://github.com/socketio/engine.io) and [karma](https://github.com/karma-runner/karma). These dependencies needed to be updated together. Updates `engine.io` from

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4450: Bump terser and @angular-devkit/build-angular in /zeppelin-web-angular

2022-08-24 Thread GitBox
dependabot[bot] opened a new pull request, #4450: URL: https://github.com/apache/zeppelin/pull/4450 Bumps [terser](https://github.com/terser/terser) and [@angular-devkit/build-angular](https://github.com/angular/angular-cli). These dependencies needed to be updated together. Updates

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4449: Bump engine.io and karma in /zeppelin-web

2022-08-24 Thread GitBox
dependabot[bot] opened a new pull request, #4449: URL: https://github.com/apache/zeppelin/pull/4449 Bumps [engine.io](https://github.com/socketio/engine.io) and [karma](https://github.com/karma-runner/karma). These dependencies needed to be updated together. Updates `engine.io` from

[GitHub] [zeppelin] jongyoul merged pull request #4442: Bump hadoop-common from 3.2.3 to 3.2.4 in /alluxio

2022-08-24 Thread GitBox
jongyoul merged PR #4442: URL: https://github.com/apache/zeppelin/pull/4442 -- 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 #4447: [Hotfix] Typo in shiro_authentication.md

2022-08-24 Thread GitBox
jongyoul merged PR #4447: URL: https://github.com/apache/zeppelin/pull/4447 -- 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 #4448: [ZEPPELIN-5802][MINOR] fix rimraf version

2022-08-24 Thread GitBox
jongyoul commented on PR #4448: URL: https://github.com/apache/zeppelin/pull/4448#issuecomment-1225524145 @kkoldduck2 Could you please handle the review? -- 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

[GitHub] [zeppelin] kkoldduck2 opened a new pull request, #4448: [ZEPPELIN-5802][MINOR] fix rimraf version

2022-08-23 Thread GitBox
kkoldduck2 opened a new pull request, #4448: URL: https://github.com/apache/zeppelin/pull/4448 ### What is this PR for? In the [ZEPPELIN-5786] Remove sap interpreter pr, the rimraf version overrided from 3.0.2 to 2.5.4 unintentionally. Therefore it needs to be fixed back to 3.0.2

[GitHub] [zeppelin] Reamer merged pull request #4446: [ZEPPELIN-5801] Add documentation for Configuration Storage

2022-08-22 Thread GitBox
Reamer merged PR #4446: URL: https://github.com/apache/zeppelin/pull/4446 -- 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 #4399: Bump commons-configuration2 from 2.7 to 2.8.0

2022-08-22 Thread GitBox
Reamer commented on PR #4399: URL: https://github.com/apache/zeppelin/pull/4399#issuecomment-1222158361 @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 comment.

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

2022-08-22 Thread GitBox
tangzhankun commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1222138790 @Samrat002 any updates on this? -- 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] Reamer commented on pull request #4444: [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-22 Thread GitBox
Reamer commented on PR #: URL: https://github.com/apache/zeppelin/pull/#issuecomment-1222115351 PR: https://github.com/apache/zeppelin/pull/4447 -- 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] Reamer opened a new pull request, #4447: [Hotfix] Typo in shiro_authentication.md

2022-08-22 Thread GitBox
Reamer opened a new pull request, #4447: URL: https://github.com/apache/zeppelin/pull/4447 ### What is this PR for? Remove one `ldapRealm.contextFactory.authenticationMechanism` ### What type of PR is it? Documentation ### Questions: * Does the licenses files need

[GitHub] [zeppelin] Reamer commented on pull request #4444: [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-22 Thread GitBox
Reamer commented on PR #: URL: https://github.com/apache/zeppelin/pull/#issuecomment-1222110447 With this PR we have defined the same property twice. `ldapRealm.contextFactory.authenticationMechanism` -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] jongyoul merged pull request #4444: [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-17 Thread GitBox
jongyoul merged PR #: URL: https://github.com/apache/zeppelin/pull/ -- 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 #4444: [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-17 Thread GitBox
jongyoul commented on PR #: URL: https://github.com/apache/zeppelin/pull/#issuecomment-1219004269 @paul8263 Yes, correct. I just would like to execute `license-check`. Please don't care about others. :-) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] paul8263 commented on pull request #4444: [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-17 Thread GitBox
paul8263 commented on PR #: URL: https://github.com/apache/zeppelin/pull/#issuecomment-1218903491 Hi @jongyoul , Coul you please help rerun the tests please? This PR is only for docs and leave those test unchanged. They should not fail. Thanks. -- This is an automated

[GitHub] [zeppelin] zkytech opened a new pull request, #4446: [ZEPPELIN-5801] Add documentation for Configuration Storage

2022-08-16 Thread GitBox
zkytech opened a new pull request, #4446: URL: https://github.com/apache/zeppelin/pull/4446 ### What is this PR for? Add documentation for Configuration Storage ### What type of PR is it? Documentation ### What is the Jira issue?

[GitHub] [zeppelin] huage1994 commented on pull request #4445: [ZEPPELIN-5800] upgrade version of apache commons configuration2 to 2.8

2022-08-16 Thread GitBox
huage1994 commented on PR #4445: URL: https://github.com/apache/zeppelin/pull/4445#issuecomment-1216389926 It's duplicated -- 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] huage1994 closed pull request #4445: [ZEPPELIN-5800] upgrade version of apache commons configuration2 to 2.8

2022-08-16 Thread GitBox
huage1994 closed pull request #4445: [ZEPPELIN-5800] upgrade version of apache commons configuration2 to 2.8 URL: https://github.com/apache/zeppelin/pull/4445 -- 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

[GitHub] [zeppelin] huage1994 opened a new pull request, #4445: [ZEPPELIN-5800] upgrade version of apache commons configuration2 to 2.8

2022-08-16 Thread GitBox
huage1994 opened a new pull request, #4445: URL: https://github.com/apache/zeppelin/pull/4445 ### What is this PR for? This PR is to upgrade version of commons-configuration from 2.7 to 2.8 due to

[GitHub] [zeppelin] paul8263 opened a new pull request, #4444: Enriched LdapRealm configuration in Shiro Authentication and fixed so…

2022-08-15 Thread GitBox
paul8263 opened a new pull request, #: URL: https://github.com/apache/zeppelin/pull/ …me wrong config items. ### What is this PR for? Changed the configuration in LdapRealm example from `ldapRealm.contextFactory.systemAuthenticationMechanism=simple` to

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

2022-08-15 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945385230 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-15 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945385230 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-15 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945391927 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-15 Thread GitBox
zkytech commented on PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#issuecomment-1214710538 I have found a new bug in this file. message data like this will fail with error `com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected a string but was

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

2022-08-14 Thread GitBox
jongyoul commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945427628 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -23,13 +23,7 @@ import java.io.IOException; import java.io.StringWriter; import

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

2022-08-14 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945426808 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -180,6 +215,9 @@ protected List parseSQLJsonOutput(String output) { List rows =

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

2022-08-14 Thread GitBox
zjffdu commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945394387 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -180,6 +215,9 @@ protected List parseSQLJsonOutput(String output) { List rows =

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

2022-08-14 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945391927 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-14 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945385230 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-14 Thread GitBox
zkytech commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945385230 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line,

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

2022-08-14 Thread GitBox
zjffdu commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945299770 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -114,6 +110,40 @@ public void open() throws InterpreterException { @Override

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

2022-08-14 Thread GitBox
zjffdu commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945299699 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -114,6 +110,40 @@ public void open() throws InterpreterException { @Override

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

2022-08-14 Thread GitBox
zjffdu commented on code in PR #4443: URL: https://github.com/apache/zeppelin/pull/4443#discussion_r945299651 ## livy/src/main/java/org/apache/zeppelin/livy/LivySparkSQLInterpreter.java: ## @@ -141,6 +171,10 @@ public InterpreterResult interpret(String line, InterpreterContext

[GitHub] [zeppelin] zkytech opened a new pull request, #4443: [ZEPPELIN-5798] Add multiline sql statement support for Livy Spark SQL interpreter

2022-08-14 Thread GitBox
zkytech opened a new pull request, #4443: URL: https://github.com/apache/zeppelin/pull/4443 ### What is this PR for? Add multiline sql statement support for Livy Spark SQL interpreter ### What type of PR is it? Feature ### What is the Jira issue?

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

2022-08-12 Thread GitBox
Reamer commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1212769264 I would like to test this beforehand in my local K8s cluster, but I am still on vacation right now. Please have a little patience. -- This is an automated message from the Apache Git

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4442: Bump hadoop-common from 3.2.3 to 3.2.4 in /alluxio

2022-08-11 Thread GitBox
dependabot[bot] opened a new pull request, #4442: URL: https://github.com/apache/zeppelin/pull/4442 Bumps hadoop-common from 3.2.3 to 3.2.4. [![Dependabot compatibility

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

2022-08-11 Thread GitBox
zlosim commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1212240313 Hi @zjffdu , I think this PR is ready to merge unless there are any other requests -- This is an automated message from the Apache Git Service. To respond to the message, please log on

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

2022-08-11 Thread GitBox
tangzhankun commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1211798308 @Samrat002 thanks a lot! I am not an expert as well. So please continue as your plan. Let me know if you have any blockers. I will try to help. -- This is an automated message

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

2022-08-10 Thread GitBox
zjffdu closed pull request #4423: [ZEPPELIN-5781]Add cross datasource query support to Spark SQL interpreter URL: https://github.com/apache/zeppelin/pull/4423 -- 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

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

2022-08-10 Thread GitBox
zkytech commented on PR #4423: URL: https://github.com/apache/zeppelin/pull/4423#issuecomment-1211460515 > @zkytech Do you have any plan for the next step? 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

[GitHub] [zeppelin] HYEEWON commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-10 Thread GitBox
HYEEWON commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r942303066 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf": "^3.0.2",

[GitHub] [zeppelin] HYEEWON commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-10 Thread GitBox
HYEEWON commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r942303066 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf": "^3.0.2",

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

2022-08-09 Thread GitBox
zjffdu commented on PR #4423: URL: https://github.com/apache/zeppelin/pull/4423#issuecomment-1210060899 @zkytech Do you have any plan for the next step? -- 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] zjffdu commented on pull request #4398: [ZEPPELIN-5760] fix passing configs to spark in k8s

2022-08-09 Thread GitBox
zjffdu commented on PR #4398: URL: https://github.com/apache/zeppelin/pull/4398#issuecomment-1210060380 @zlosim 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] Samrat002 commented on pull request #4400: ZEPPELIN-3527. replace log4j with reload4j

2022-08-09 Thread GitBox
Samrat002 commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1209467423 @tangzhankun I got occupied on my current company deliverables , wasnt able to make much of progress ! i am expecting to get quality time for this task from next week. If this is

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

2022-08-09 Thread GitBox
tangzhankun commented on PR #4400: URL: https://github.com/apache/zeppelin/pull/4400#issuecomment-1209216348 @Samrat002 thanks for the help. This is a critical fix. Any update on this? Seems the PR is outdated. -- This is an automated message from the Apache Git Service. To respond to

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

2022-08-07 Thread GitBox
jongyoul commented on PR #4441: URL: https://github.com/apache/zeppelin/pull/4441#issuecomment-1207666015 BTW, I've checked the code and code looks finding the version of Livy so I think we should change/add some code for supporting a newer version of livy. @soicem Could you please check

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

2022-08-07 Thread GitBox
jongyoul commented on PR #4441: URL: https://github.com/apache/zeppelin/pull/4441#issuecomment-1207665649 > It looks like spark integration is failed, not sure whether it is related to this PR Sometimes, it fails so ... -- This is an automated message from the Apache Git Service.

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

2022-08-07 Thread GitBox
zjffdu commented on PR #4441: URL: https://github.com/apache/zeppelin/pull/4441#issuecomment-1207569378 It looks like spark integration is failed, not sure whether it is related to this PR -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [zeppelin] soicem opened a new pull request, #4441: [ZEPPELIN-4871] Fix livy interpreter 0.7.1 donwload link

2022-08-07 Thread GitBox
soicem opened a new pull request, #4441: URL: https://github.com/apache/zeppelin/pull/4441 ### What is this PR for? During a previous [PR](https://github.com/apache/zeppelin/pull/4433) related to livy interpreter version upgrade, CI is not working because of unvalid download link of

[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4440: Bump postgresql from 42.3.3 to 42.4.1 in /jdbc

2022-08-06 Thread GitBox
dependabot[bot] opened a new pull request, #4440: URL: https://github.com/apache/zeppelin/pull/4440 Bumps [postgresql](https://github.com/pgjdbc/pgjdbc) from 42.3.3 to 42.4.1. Release notes Sourced from https://github.com/pgjdbc/pgjdbc/releases;>postgresql's releases.

[GitHub] [zeppelin] maziyarpanahi opened a new pull request, #4439: [ZEPPELIN-5191] Fix the wrong Docker image name in zeppelin-server.yaml

2022-08-04 Thread GitBox
maziyarpanahi opened a new pull request, #4439: URL: https://github.com/apache/zeppelin/pull/4439 - There has never been any Docker image named `zeppelin-interpreter` or `zeppelin-server`. - The latest version also should be 0.10.1 ### What is this PR for? A few sentences

[GitHub] [zeppelin] kkoldduck2 commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-03 Thread GitBox
kkoldduck2 commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r937250589 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf":

[GitHub] [zeppelin] jongyoul merged pull request #4381: Bump gson from 2.2 to 2.8.9 in /shell

2022-08-03 Thread GitBox
jongyoul merged PR #4381: URL: https://github.com/apache/zeppelin/pull/4381 -- 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] kkoldduck2 commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-03 Thread GitBox
kkoldduck2 commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r937242091 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf":

[GitHub] [zeppelin] kkoldduck2 commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-03 Thread GitBox
kkoldduck2 commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r937242091 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf":

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-03 Thread GitBox
jongyoul commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r936807226 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf": "^3.0.2",

[GitHub] [zeppelin] HYEEWON commented on a diff in pull request #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-03 Thread GitBox
HYEEWON commented on code in PR #4437: URL: https://github.com/apache/zeppelin/pull/4437#discussion_r936550037 ## zeppelin-web/package.json: ## @@ -93,7 +93,7 @@ "postcss-loader": "^3.0.0", "protractor": "^5.4.1", "raw-loader": "^0.5.1", -"rimraf": "^3.0.2",

[GitHub] [zeppelin] jongyoul merged pull request #4434: [ZEPPELIN-5785] Remove ksql interpreter

2022-08-03 Thread GitBox
jongyoul merged PR #4434: URL: https://github.com/apache/zeppelin/pull/4434 -- 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 #4402: [ZEPPELIN-5762] Fix default profile activation option to build succes…

2022-08-03 Thread GitBox
jongyoul commented on PR #4402: URL: https://github.com/apache/zeppelin/pull/4402#issuecomment-1203539560 @YeonghyeonKO FYI, It's already been merged by #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

[GitHub] [zeppelin] jongyoul merged pull request #4427: [ZEPPELIN-5638] Add flink job interval check

2022-08-02 Thread GitBox
jongyoul merged PR #4427: URL: https://github.com/apache/zeppelin/pull/4427 -- 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 #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-08-02 Thread GitBox
jongyoul merged PR #4438: URL: https://github.com/apache/zeppelin/pull/4438 -- 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 #4437: [ZEPPELIN-5786] Remove sap interpreter

2022-08-02 Thread GitBox
jongyoul merged PR #4437: URL: https://github.com/apache/zeppelin/pull/4437 -- 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] gbnam commented on pull request #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-08-01 Thread GitBox
gbnam commented on PR #4438: URL: https://github.com/apache/zeppelin/pull/4438#issuecomment-1201925701 Removed `hbase-server` dependency from hbase interpreter. Please check the commits. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] soicem commented on pull request #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-08-01 Thread GitBox
soicem commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1200835978 @jongyoul OK, I will create another PR and fix the issue soon ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [zeppelin] jongyoul merged pull request #4436: [ZEPPELIN-5789] Update Dockerfile to use JDK instead of JRE

2022-07-31 Thread GitBox
jongyoul merged PR #4436: URL: https://github.com/apache/zeppelin/pull/4436 -- 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 #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-31 Thread GitBox
jongyoul commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1200735324 @soicem I missed checking the result of CI. Could you please make another PR to fix the issue? ``` + echo '.livy-dist does not have livy-0.7.1-incubating-bin downloading ...'

[GitHub] [zeppelin] jongyoul commented on pull request #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-07-31 Thread GitBox
jongyoul commented on PR #4438: URL: https://github.com/apache/zeppelin/pull/4438#issuecomment-1200732148 Thank you for checking it. Let's see how it goes on :-). BTW, `hbase-server` might not be used by the code level so I guess we can remove the dependency itself. -- This is an

[GitHub] [zeppelin] gbnam commented on pull request #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-07-31 Thread GitBox
gbnam commented on PR #4438: URL: https://github.com/apache/zeppelin/pull/4438#issuecomment-1200542377 I solved error by exclusion glassfish. Please let me know if there is anything else I should do. Thanks! -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] jongyoul merged pull request #4435: [ZEPPELIN-5788] Remove beam interpreter

2022-07-31 Thread GitBox
jongyoul merged PR #4435: URL: https://github.com/apache/zeppelin/pull/4435 -- 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 #4428: [ZEPPELIN-5783] Remove Ignite Interpreter

2022-07-31 Thread GitBox
jongyoul merged PR #4428: URL: https://github.com/apache/zeppelin/pull/4428 -- 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 #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-31 Thread GitBox
jongyoul merged PR #4433: URL: https://github.com/apache/zeppelin/pull/4433 -- 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] gbnam commented on pull request #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-07-29 Thread GitBox
gbnam commented on PR #4438: URL: https://github.com/apache/zeppelin/pull/4438#issuecomment-1200079589 Error occured when maven download some libs blocked http request. I solved it by editing the maven settings in local env.

[GitHub] [zeppelin] soicem commented on pull request #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-29 Thread GitBox
soicem commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1200078859 @zjffdu @jongyoul Oh, I changed it right away. Thanks for the feedback! :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [zeppelin] jongyoul commented on pull request #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-29 Thread GitBox
jongyoul commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1200067995 @soicem Could you please handle the comment by @zjffdu ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [zeppelin] jongyoul commented on pull request #4432: [ZEPPELIN-5783] Remove kylin interpreter

2022-07-29 Thread GitBox
jongyoul commented on PR #4432: URL: https://github.com/apache/zeppelin/pull/4432#issuecomment-1200067339 It's related to ZEPPELIN-5786 -- 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] jongyoul commented on pull request #4438: [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12

2022-07-29 Thread GitBox
jongyoul commented on PR #4438: URL: https://github.com/apache/zeppelin/pull/4438#issuecomment-1200066216 Could you please check what happens when you building it? ``` [INFO] Zeppelin: Angular interpreter .. SUCCESS [ 0.375 s] [INFO] Zeppelin: Livy

[GitHub] [zeppelin] gbnam opened a new pull request, #4438: Update hbase interpreter version from 1.0.0 to 2.4.12

2022-07-29 Thread GitBox
gbnam opened a new pull request, #4438: URL: https://github.com/apache/zeppelin/pull/4438 ### What is this PR for? Update hbase interpreter version from 1.0.0 to 2.4.12 ### What type of PR is it? Refactoring ### Todos [v] update hbase interpreter version ###

[GitHub] [zeppelin] zjffdu commented on pull request #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-29 Thread GitBox
zjffdu commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1199445353 Another place need to update, https://github.com/apache/zeppelin/blob/master/.github/workflows/core.yml#L407 -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] jongyoul merged pull request #4431: [ZEPPELIN-4794] Add comment about changing localhost when running Zeppelin on WSL or Windows

2022-07-29 Thread GitBox
jongyoul merged PR #4431: URL: https://github.com/apache/zeppelin/pull/4431 -- 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 #4430: [ZEPPELIN-5782] Remove geode interpreter

2022-07-29 Thread GitBox
jongyoul merged PR #4430: URL: https://github.com/apache/zeppelin/pull/4430 -- 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 #4436: [ZEPPELIN-5789] Update Dockerfile to use JDK instead of JRE

2022-07-29 Thread GitBox
Reamer commented on PR #4436: URL: https://github.com/apache/zeppelin/pull/4436#issuecomment-1199212303 > I was only able to set the target version to 0.11 in Jira... Is there a way to request inclusion in any further 0.10.x release if it happens? Or I presume that will happen by default

[GitHub] [zeppelin] soicem commented on pull request #4433: [ZEPPELIN-4871] Update Livy interpreter to 0.7.1

2022-07-28 Thread GitBox
soicem commented on PR #4433: URL: https://github.com/apache/zeppelin/pull/4433#issuecomment-1198107226 @zjffdu thanks for letting me know :) I changed it right away ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

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