[GitHub] [zeppelin] HYEEWON opened a new pull request, #4424: [ZEPPELIN-5779] Remove pig interpreter

2022-07-22 Thread GitBox
HYEEWON opened a new pull request, #4424: URL: https://github.com/apache/zeppelin/pull/4424 ### What is this PR for? * Remove pig interpreter ### What type of PR is it? Refactoring ### Todos * Remove pig and related things ### What is the Jira issue? [ZEPPELIN

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

2022-07-22 Thread GitBox
jongyoul commented on PR #4423: URL: https://github.com/apache/zeppelin/pull/4423#issuecomment-1193072604 Hello, thank you for the contribution. By the way, I have a question. IIUC, You can already use several data sources for Spark. Could you please the benefit of this way? It looks changi

[GitHub] [zeppelin] zkytech opened a new pull request, #4423: [ZEPPELIN-5781]Add cross datasource query support to Spark SQL interpreter

2022-07-22 Thread GitBox
zkytech opened a new pull request, #4423: URL: https://github.com/apache/zeppelin/pull/4423 ### What is this PR for? Add **cross datasource query** support to Spark SQL interpreter, currently support cross datasource query with these datasources: - hive - jdbc - mongodb

[jira] [Created] (ZEPPELIN-5781) Add cross datasource query support to Spark SQL interpreter

2022-07-22 Thread Kunyuan Zhang (Jira)
Kunyuan Zhang created ZEPPELIN-5781: --- Summary: Add cross datasource query support to Spark SQL interpreter Key: ZEPPELIN-5781 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5781 Project: Zeppel

[GitHub] [zeppelin] JoWonHyeung commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
JoWonHyeung commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927675104 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/conf/ZeppelinConfigurationTest.java: ## @@ -69,18 +69,18 @@ public void getAllowedOriginsNoneTest() throws Ma

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

2022-07-22 Thread GitBox
ganlute commented on PR #4410: URL: https://github.com/apache/zeppelin/pull/4410#issuecomment-1192692807 > Will merge it soon. oh that is great, thank you~ šŸ˜„ -- 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] JoWonHyeung commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
JoWonHyeung commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927675104 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/conf/ZeppelinConfigurationTest.java: ## @@ -69,18 +69,18 @@ public void getAllowedOriginsNoneTest() throws Ma

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
Reamer commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927592543 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +637,9 @@ public String getInterpreterPortRange() { return g

[GitHub] [zeppelin] Reamer opened a new pull request, #4422: [ZEPPELIN-5780] Unify paragraph errors

2022-07-22 Thread GitBox
Reamer opened a new pull request, #4422: URL: https://github.com/apache/zeppelin/pull/4422 ### What is this PR for? This PR fixes the non-equal behavior of the InterpreterNotFoundException. It is tried to avoid duplicate codes. ### What type of PR is it? Bug Fix ### W

[jira] [Created] (ZEPPELIN-5780) Running a paragraph with an interpreter that is not present will result in the Ready state

2022-07-22 Thread Philipp Dallig (Jira)
Philipp Dallig created ZEPPELIN-5780: Summary: Running a paragraph with an interpreter that is not present will result in the Ready state Key: ZEPPELIN-5780 URL: https://issues.apache.org/jira/browse/ZEPPELIN-

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
jongyoul commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927577081 ## zeppelin-zengine/src/test/java/org/apache/zeppelin/conf/ZeppelinConfigurationTest.java: ## @@ -69,18 +69,18 @@ public void getAllowedOriginsNoneTest() throws Malfo

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
jongyoul commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927575453 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +637,9 @@ public String getInterpreterPortRange() { return

[GitHub] [zeppelin] Reamer merged pull request #4421: HOTFIX - fix s3 enforcer

2022-07-22 Thread GitBox
Reamer merged PR #4421: URL: https://github.com/apache/zeppelin/pull/4421 -- 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, #4421: HOTFIX - fix s3 enforcer

2022-07-22 Thread GitBox
Reamer opened a new pull request, #4421: URL: https://github.com/apache/zeppelin/pull/4421 ### What is this PR for? This small PR fix the enforcer plugin. ### What type of PR is it? - Hot Fix ### How should this be tested? * CI ### Questions: * Does the licens

[GitHub] [zeppelin] jongyoul merged pull request #4420: [HOTFIX] [ZEPPELIN-3945] Remove blank in .gitattributes

2022-07-22 Thread GitBox
jongyoul merged PR #4420: URL: https://github.com/apache/zeppelin/pull/4420 -- 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 #4420: [HOTFIX] [ZEPPELIN-3945] Remove blank in .gitattributes

2022-07-22 Thread GitBox
jongyoul commented on PR #4420: URL: https://github.com/apache/zeppelin/pull/4420#issuecomment-1192405748 I'll merge it now as it breaks user behaviors 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

[GitHub] [zeppelin] JoWonHyeung commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
JoWonHyeung commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927499221 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +633,12 @@ public String getInterpreterPortRange() { re

[GitHub] [zeppelin] proceane opened a new pull request, #4420: Remove blank in .gitattributes

2022-07-22 Thread GitBox
proceane opened a new pull request, #4420: URL: https://github.com/apache/zeppelin/pull/4420 ### What is this PR for? The eof setting of gitattributes was applied to all files including `.js`, so there was a problem that the file was changed in the Windows environment. So, I removed th

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
jongyoul commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927495792 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +633,12 @@ public String getInterpreterPortRange() { retur

[GitHub] [zeppelin] JoWonHyeung commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
JoWonHyeung commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927495429 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +633,12 @@ public String getInterpreterPortRange() { re

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
jongyoul commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927494204 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -636,8 +633,12 @@ public String getInterpreterPortRange() { retur

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration class

2022-07-22 Thread GitBox
jongyoul commented on code in PR #4419: URL: https://github.com/apache/zeppelin/pull/4419#discussion_r927492092 ## zeppelin-interpreter/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java: ## @@ -25,11 +25,7 @@ import java.net.URISyntaxException; import java.time

[GitHub] [zeppelin] JoWonHyeung opened a new pull request, #4419: [ZEPPELIN-5773] Fix isWindowsPath Function in ZeppelinConfiguration cā€¦

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

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

2022-07-22 Thread GitBox
jongyoul merged PR #4407: URL: https://github.com/apache/zeppelin/pull/4407 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apa

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

2022-07-22 Thread GitBox
jongyoul merged PR #4412: URL: https://github.com/apache/zeppelin/pull/4412 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apa

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

2022-07-22 Thread GitBox
jongyoul merged PR #4418: URL: https://github.com/apache/zeppelin/pull/4418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apa