[GitHub] [calcite] jacques-n opened a new pull request #2653: [CALCITE-4952] (Re)introduce a simplistic RelMetadataQuery option

2021-12-18 Thread GitBox
jacques-n opened a new pull request #2653: URL: https://github.com/apache/calcite/pull/2653 - Introduce new ProxyingMetadataHandlerProvider that avoids code compilation - Update RelMetadataTest to factor out metadata handling from tests - Introduce tests for both Janino and the proxyin

[GitHub] [calcite] rubenada commented on pull request #2652: Set timeout for running Druid tests in GitHub CI

2021-12-18 Thread GitBox
rubenada commented on pull request #2652: URL: https://github.com/apache/calcite/pull/2652#issuecomment-997285574 Sounds reasonable. 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 spec

[GitHub] [calcite] zabetak opened a new pull request #2652: Set timeout for running Druid tests in GitHub CI

2021-12-18 Thread GitBox
zabetak opened a new pull request #2652: URL: https://github.com/apache/calcite/pull/2652 When tests run successfully they don't take more than a few minutes. Sometimes though they get stuck and run forever (till the max GitHub timeout of 6h) so set a timeout to 10minutes to avoid wast

[GitHub] [calcite] julianhyde commented on pull request #2628: [CALCITE-4917] Add test for 'IS NOT NULL(a) AND a=b' simplification

2021-12-18 Thread GitBox
julianhyde commented on pull request #2628: URL: https://github.com/apache/calcite/pull/2628#issuecomment-997275042 @asolimando You should announce your intentions on the '[Calcite Release] Preparing for 1.29.0 Release' thread. As RM, Rui is about to lock down master branch. -- This is

[GitHub] [calcite] asolimando commented on pull request #2628: [CALCITE-4917] Add test for 'IS NOT NULL(a) AND a=b' simplification

2021-12-18 Thread GitBox
asolimando commented on pull request #2628: URL: https://github.com/apache/calcite/pull/2628#issuecomment-997267203 If there are no objections I will merge in the next 24 hours -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[calcite-avatica] branch master updated: [CALCITE-4951] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica.git The following commit(s) were added to refs/heads/master by this push: new 8a20480 [CALCITE-4951] Upgrade log4j2

[GitHub] [calcite-avatica] zabetak merged pull request #168: [CALCITE-4951] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread GitBox
zabetak merged pull request #168: URL: https://github.com/apache/calcite-avatica/pull/168 -- 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: commits-

[calcite] branch master updated: [CALCITE-4950] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new 0d17965 [CALCITE-4950] Upgrade log4j2 version

[GitHub] [calcite] zabetak merged pull request #2651: [CALCITE-4950] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread GitBox
zabetak merged pull request #2651: URL: https://github.com/apache/calcite/pull/2651 -- 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: commits-unsubs

[GitHub] [calcite] whuGuo commented on pull request #2611: [CALCITE-4887] Filter subQuery remove not fully optimized

2021-12-18 Thread GitBox
whuGuo commented on pull request #2611: URL: https://github.com/apache/calcite/pull/2611#issuecomment-997211717 > @whuGuo, if you want to retest do as follows: > > ``` > In the special case, that the CI build failed, and the failure is not caused by your changes create an empty co

[GitHub] [calcite] asolimando edited a comment on pull request #2611: [CALCITE-4887] Filter subQuery remove not fully optimized

2021-12-18 Thread GitBox
asolimando edited a comment on pull request #2611: URL: https://github.com/apache/calcite/pull/2611#issuecomment-997210518 @whuGuo, if you want to retest do as follows: ``` In the special case, that the CI build failed, and the failure is not caused by your changes create an empty com

[GitHub] [calcite] asolimando commented on pull request #2611: [CALCITE-4887] Filter subQuery remove not fully optimized

2021-12-18 Thread GitBox
asolimando commented on pull request #2611: URL: https://github.com/apache/calcite/pull/2611#issuecomment-997210518 @whuGuo, if you want to retest do as follows: ``` In the special case, that the CI build failed, and the failure is not caused by your changes create an empty commit (gi

[GitHub] [calcite-avatica] zabetak opened a new pull request #168: [CALCITE-4951] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread GitBox
zabetak opened a new pull request #168: URL: https://github.com/apache/calcite-avatica/pull/168 The 2.17.0 release fixes a new security vulnerability CVE-2021-45105. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [calcite] zabetak opened a new pull request #2651: [CALCITE-4950] Upgrade log4j2 version to 2.17.0

2021-12-18 Thread GitBox
zabetak opened a new pull request #2651: URL: https://github.com/apache/calcite/pull/2651 The 2.17.0 release fixes a new security vulnerability CVE-2021-45105. -- 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] [calcite] zabetak closed pull request #2469: [CALCITE-4704] Log plan on rule application using explain formatting

2021-12-18 Thread GitBox
zabetak closed pull request #2469: URL: https://github.com/apache/calcite/pull/2469 -- 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: commits-unsubs

[calcite] branch master updated: [CALCITE-4704] Log plan on rule application using explain formatting

2021-12-18 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new e42b85a [CALCITE-4704] Log plan on rule appli

[GitHub] [calcite] NobiGo merged pull request #2464: [CALCITE-4700] AggregateUnionTransposeRule produces wrong group sets for the top Aggregate (Vladimir Ozerov)

2021-12-18 Thread GitBox
NobiGo merged pull request #2464: URL: https://github.com/apache/calcite/pull/2464 -- 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: commits-unsubsc

[calcite] branch master updated: [CALCITE-4700] AggregateUnionTransposeRule produces wrong group sets for the top Aggregate (Vladimir Ozerov)

2021-12-18 Thread xiong
This is an automated email from the ASF dual-hosted git repository. xiong pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new 538016e [CALCITE-4700] AggregateUnionTransposeR

[GitHub] [calcite] whuGuo removed a comment on pull request #2611: [CALCITE-4887] Filter subQuery remove not fully optimized

2021-12-18 Thread GitBox
whuGuo removed a comment on pull request #2611: URL: https://github.com/apache/calcite/pull/2611#issuecomment-997181513 retest this please -- 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] [calcite] whuGuo commented on pull request #2611: [CALCITE-4887] Filter subQuery remove not fully optimized

2021-12-18 Thread GitBox
whuGuo commented on pull request #2611: URL: https://github.com/apache/calcite/pull/2611#issuecomment-997181513 retest this please -- 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 co

[GitHub] [calcite] ILuffZhe commented on a change in pull request #2650: [CALCITE-4949] Evaluate if using org.codelibs is appropriate for an Elasticsearch painless dependency

2021-12-18 Thread GitBox
ILuffZhe commented on a change in pull request #2650: URL: https://github.com/apache/calcite/pull/2650#discussion_r771799669 ## File path: elasticsearch/build.gradle.kts ## @@ -37,6 +37,9 @@ dependencies { testImplementation("org.apache.logging.log4j:log4j-api") tes

[GitHub] [calcite] francesco-gini88 commented on a change in pull request #2620: [CALCITE-4907] JdbcJoin does not convert Join with always true/false …

2021-12-18 Thread GitBox
francesco-gini88 commented on a change in pull request #2620: URL: https://github.com/apache/calcite/pull/2620#discussion_r771798236 ## File path: core/src/test/java/org/apache/calcite/test/JdbcAdapterTest.java ## @@ -50,23 +50,25 @@ * same time. */ private static final

[GitHub] [calcite] NobiGo commented on a change in pull request #2650: [CALCITE-4949] Evaluate if using org.codelibs is appropriate for an Elasticsearch painless dependency

2021-12-18 Thread GitBox
NobiGo commented on a change in pull request #2650: URL: https://github.com/apache/calcite/pull/2650#discussion_r771798086 ## File path: elasticsearch/build.gradle.kts ## @@ -37,6 +37,9 @@ dependencies { testImplementation("org.apache.logging.log4j:log4j-api") testI