[GitHub] [zeppelin] zhugezifang commented on pull request #4598: [ZEPPELIN-5896]feat:add sql debug feature

2023-05-10 Thread via GitHub
zhugezifang commented on PR #4598: URL: https://github.com/apache/zeppelin/pull/4598#issuecomment-1543339056 hi @Reamer is this pr has other problems? i really want to add this feature to zeppelin, could you help me complete it? -- This is an automated message from the Apache Git Service

[jira] [Created] (ZEPPELIN-5911) RemoteInterpreterTest can not run in window environment

2023-05-10 Thread duanxiaoqiu (Jira)
duanxiaoqiu created ZEPPELIN-5911: - Summary: RemoteInterpreterTest can not run in window environment Key: ZEPPELIN-5911 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5911 Project: Zeppelin

[GitHub] [zeppelin] Reamer commented on pull request #4595: [ZEPPELIN-5907][JUnit 5 Migration] Module zeppelin-interpreter

2023-05-10 Thread via GitHub
Reamer commented on PR #4595: URL: https://github.com/apache/zeppelin/pull/4595#issuecomment-1541824077 > @Reamer I would like to refactor zeppelin-interpreter package. Is it possible to do it from now on? Yes I think so, I just have little things in the pipeline. -- This is an aut

[GitHub] [zeppelin] jongyoul commented on pull request #4595: [ZEPPELIN-5907][JUnit 5 Migration] Module zeppelin-interpreter

2023-05-10 Thread via GitHub
jongyoul commented on PR #4595: URL: https://github.com/apache/zeppelin/pull/4595#issuecomment-1541746147 @Reamer I would like to refactor zeppelin-interpreter package. Is it possible to do it from now on? -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [zeppelin] Reamer commented on pull request #4600: [ZEPPELIN-5850] Migrate zeppelin-server to Junit5

2023-05-10 Thread via GitHub
Reamer commented on PR #4600: URL: https://github.com/apache/zeppelin/pull/4600#issuecomment-1541515926 This PR depends on #4601. -- 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 comm

[GitHub] [zeppelin] zhugezifang commented on pull request #4598: [ZEPPELIN-5896]feat:add sql debug feature

2023-05-10 Thread via GitHub
zhugezifang commented on PR #4598: URL: https://github.com/apache/zeppelin/pull/4598#issuecomment-1541515755 hi @Reamer ,is there other problem of this pr? could help me review it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [zeppelin] Reamer opened a new pull request, #4601: [ZEPPELIN-5910] Remove hamcrest-all as compile dependency

2023-05-10 Thread via GitHub
Reamer opened a new pull request, #4601: URL: https://github.com/apache/zeppelin/pull/4601 ### What is this PR for? This PR is minimally invasive. This PR updates atomix from 3.0.0-rc4 to 3.0.0-rc5m which removes hamcrest-all as compile dependency from zeppelin-interpreter. https

[GitHub] [zeppelin] xieyi888 commented on pull request #4597: [ZEPPELIN-5908]JDBCSecurityImpl should relogin when authType='kerberos' to void kerberos ticket expired

2023-05-10 Thread via GitHub
xieyi888 commented on PR #4597: URL: https://github.com/apache/zeppelin/pull/4597#issuecomment-1541506278 > LGTM Unfortunately, I cannot verify the change, but they look plausible. I will merge the change on Friday as long as no further comments are received. thanks for your review

[jira] [Created] (ZEPPELIN-5910) zeppelin-interpreter has hamcrest-all as compile dependency

2023-05-10 Thread Philipp Dallig (Jira)
Philipp Dallig created ZEPPELIN-5910: Summary: zeppelin-interpreter has hamcrest-all as compile dependency Key: ZEPPELIN-5910 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5910 Project: Zepp

[GitHub] [zeppelin] Reamer opened a new pull request, #4600: [ZEPPELIN-5850] Migrate zeppelin-server to Junit5

2023-05-10 Thread via GitHub
Reamer opened a new pull request, #4600: URL: https://github.com/apache/zeppelin/pull/4600 ### What is this PR for? This PR migrates the Zeppelin-server module from JUnit4 to JUnit 5. ### What type of PR is it? Improvement Test ### What is the Jira issue? * https