[GitHub] [drill] vvysotskyi merged pull request #2534: DRILL-8210: Add substring convertlet

2022-05-09 Thread GitBox
vvysotskyi merged PR #2534: URL: https://github.com/apache/drill/pull/2534 -- 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] [drill] vvysotskyi merged pull request #2537: DRILL-8213: Replace deprecated RelNode.getRows with RelNode.estimateRowCount

2022-05-09 Thread GitBox
vvysotskyi merged PR #2537: URL: https://github.com/apache/drill/pull/2537 -- 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] [drill-test-framework] agozhiy commented on pull request #1: Resurrection

2022-05-09 Thread GitBox
agozhiy commented on PR #1: URL: https://github.com/apache/drill-test-framework/pull/1#issuecomment-1121369437 Regarding the UDF tests: the test folder doesn't contain any tests so there is no difference what category to use. I searched in git history and it was always empty, not even

[GitHub] [drill-test-framework] agozhiy commented on a diff in pull request #1: Resurrection

2022-05-09 Thread GitBox
agozhiy commented on code in PR #1: URL: https://github.com/apache/drill-test-framework/pull/1#discussion_r868211411 ## framework/pom.xml: ## @@ -7,12 +7,27 @@ 1.0.0-SNAPSHOT 6.4 -1.2.17 +1.7.26 +1.2.9 2.10.0 -${env.DRILL_VERSION} -

[jira] [Created] (DRILL-8218) Add unit tests for StorageResource REST endpoints

2022-05-09 Thread James Turton (Jira)
James Turton created DRILL-8218: --- Summary: Add unit tests for StorageResource REST endpoints Key: DRILL-8218 URL: https://issues.apache.org/jira/browse/DRILL-8218 Project: Apache Drill Issue

[GitHub] [drill] cgivre merged pull request #2536: DRILL-8212: Join queries fail with StackOverflowError

2022-05-09 Thread GitBox
cgivre merged PR #2536: URL: https://github.com/apache/drill/pull/2536 -- 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...@drill.apache.org

[GitHub] [drill-test-framework] jnturton commented on a diff in pull request #1: Resurrection

2022-05-09 Thread GitBox
jnturton commented on code in PR #1: URL: https://github.com/apache/drill-test-framework/pull/1#discussion_r867959310 ## framework/pom.xml: ## @@ -7,12 +7,27 @@ 1.0.0-SNAPSHOT 6.4 -1.2.17 +1.7.26 +1.2.9 2.10.0 -${env.DRILL_VERSION} -

[GitHub] [drill-test-framework] jnturton commented on pull request #1: Resurrection

2022-05-09 Thread GitBox
jnturton commented on PR #1: URL: https://github.com/apache/drill-test-framework/pull/1#issuecomment-1121029474 Thanks for this! - Are the new modes that can be seen in the JSON config files, e.g. `rm`, `cp`, `dfs_cp`, `post_rm`, the new "data preparation phases" that you mention

Re: Bugs in DRILL-8155 currently affecting Drill master

2022-05-09 Thread James Turton
After the merger of #2540 , I'm not currently aware of any more open problems but if you do encounter one just shout. On 2022/05/07 09:27, James Turton wrote: Hi Devs DRILL-8155 seeks to expand the authentication modes supported by Drill plugins,

[GitHub] [drill] jnturton merged pull request #2540: DRILL-8215: Remove SecurityContext from PluginConfigWrapper

2022-05-09 Thread GitBox
jnturton merged PR #2540: URL: https://github.com/apache/drill/pull/2540 -- 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] [drill] jnturton commented on a diff in pull request #2540: DRILL-8215: Remove SecurityContext from PluginConfigWrapper

2022-05-09 Thread GitBox
jnturton commented on code in PR #2540: URL: https://github.com/apache/drill/pull/2540#discussion_r867783408 ## exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/CredentialResources.java: ## @@ -125,11 +125,18 @@ public List getConfigsFor(@PathParam("group")

[GitHub] [drill] jnturton merged pull request #2532: DRILL-8208: Create builder for SqlSelect

2022-05-09 Thread GitBox
jnturton merged PR #2532: URL: https://github.com/apache/drill/pull/2532 -- 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: