[calcite] branch master updated: [CALCITE-3847] Decorrelation for join with lateral table outputs wrong plan if the join condition contains correlation variables

2020-03-09 Thread danny0405
This is an automated email from the ASF dual-hosted git repository. danny0405 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 8d4820f [CALCITE-3847] Decorrelation for

[GitHub] [calcite] danny0405 merged pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron…

2020-03-09 Thread GitBox
danny0405 merged pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron… URL: https://github.com/apache/calcite/pull/1850 This is an automated message from the Apache Git Service. To

[GitHub] [calcite] vineetgarg02 commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
vineetgarg02 commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390049788 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390028555 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390036335 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390027440 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390040676 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390039710 ## File path:

[GitHub] [calcite] jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
jcamachor commented on a change in pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852#discussion_r390036118 ## File path:

[GitHub] [calcite] hsyuan commented on issue #1849: CALCITE-3815: Add missing SQL standard aggregate functions: EVERY, SO…

2020-03-09 Thread GitBox
hsyuan commented on issue #1849: CALCITE-3815: Add missing SQL standard aggregate functions: EVERY, SO… URL: https://github.com/apache/calcite/pull/1849#issuecomment-596838704 Related PR: https://github.com/apache/calcite/pull/1694

[calcite] 01/02: [CALCITE-3753] Remove rule queue importance

2020-03-09 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 80e6b023d4b92b2b25908c52e10f50792787cef8 Author: Haisheng Yuan AuthorDate: Fri Feb 28 16:01:52 2020 -0800

[calcite] branch master updated (4208d0b -> 15fa9bc)

2020-03-09 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 4208d0b [CALCITE-3823] Do not use String.replaceAll new 80e6b02 [CALCITE-3753] Remove rule queue importance

[calcite] 02/02: [CALCITE-3753] Introduce SubstitutionRule interface and execute substitutional rule first

2020-03-09 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 15fa9bc67a8ed12ec6b8e0d1dbced8760ae307ab Author: Haisheng Yuan AuthorDate: Sun Mar 8 16:04:42 2020 -0500

[GitHub] [calcite] hsyuan closed pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-09 Thread GitBox
hsyuan closed pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [calcite] vineetgarg02 opened a new pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg)

2020-03-09 Thread GitBox
vineetgarg02 opened a new pull request #1852: [CALCITE-3848] Materialized view rewriting fails for mv consisting of group by on join keys (Vineet Garg) URL: https://github.com/apache/calcite/pull/1852 This request fixes an issue in materialize view rewriting mapping logic

[GitHub] [calcite] vineetgarg02 closed pull request #1851: CALCITE-3848 Materialized view rewriting fails for mv consisting of group by on join keys

2020-03-09 Thread GitBox
vineetgarg02 closed pull request #1851: CALCITE-3848 Materialized view rewriting fails for mv consisting of group by on join keys URL: https://github.com/apache/calcite/pull/1851 This is an automated message from the Apache

[GitHub] [calcite] vineetgarg02 opened a new pull request #1851: CALCITE-3848 Materialized view rewriting fails for mv consisting of group by on join keys

2020-03-09 Thread GitBox
vineetgarg02 opened a new pull request #1851: CALCITE-3848 Materialized view rewriting fails for mv consisting of group by on join keys URL: https://github.com/apache/calcite/pull/1851 This is an automated message from the

[GitHub] [calcite] dependabot[bot] commented on issue #1577: Bump spark-core_2.10 from 2.2.0 to 2.2.2

2020-03-09 Thread GitBox
dependabot[bot] commented on issue #1577: Bump spark-core_2.10 from 2.2.0 to 2.2.2 URL: https://github.com/apache/calcite/pull/1577#issuecomment-596731250 Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the

[GitHub] [calcite] dependabot[bot] commented on issue #1578: Bump checkstyle from 7.8.2 to 8.18

2020-03-09 Thread GitBox
dependabot[bot] commented on issue #1578: Bump checkstyle from 7.8.2 to 8.18 URL: https://github.com/apache/calcite/pull/1578#issuecomment-596731251 Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update

[calcite] 03/03: [CALCITE-3823] Do not use String.replaceAll

2020-03-09 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 4208d0ba6f2a749692fe64181a1373af07d55db5 Author: Julian Hyde AuthorDate: Wed Feb 26 12:26:07 2020 -0800

[calcite] branch master updated (74536b7 -> 4208d0b)

2020-03-09 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 74536b7 [CALCITE-3838] Support Calc in

[calcite] 02/03: [CALCITE-3412] FLOOR(timestamp TO WEEK) gives wrong result

2020-03-09 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 91f5bb5589ca5b7c08b09969ad21c5e9bd18d293 Author: Julian Hyde AuthorDate: Thu Oct 17 01:22:09 2019 -0700

[calcite] 01/03: [CALCITE-3839] After calling RelBuilder.aggregate, cannot lookup field by name

2020-03-09 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit b632152f5c159334c76245b8816c721edca84fd5 Author: Julian Hyde AuthorDate: Mon Mar 2 16:29:48 2020 -0800

[GitHub] [calcite] vlsi commented on a change in pull request #1827: [CALCITE-3821] RelOptUtil::containsMultisetOrWindowedAgg doesn't real…

2020-03-09 Thread GitBox
vlsi commented on a change in pull request #1827: [CALCITE-3821] RelOptUtil::containsMultisetOrWindowedAgg doesn't real… URL: https://github.com/apache/calcite/pull/1827#discussion_r389850722 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptUtil.java ## @@

[GitHub] [calcite] xndai commented on a change in pull request #1827: [CALCITE-3821] RelOptUtil::containsMultisetOrWindowedAgg doesn't real…

2020-03-09 Thread GitBox
xndai commented on a change in pull request #1827: [CALCITE-3821] RelOptUtil::containsMultisetOrWindowedAgg doesn't real… URL: https://github.com/apache/calcite/pull/1827#discussion_r389841674 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptUtil.java ## @@

[GitHub] [calcite] hsyuan commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-09 Thread GitBox
hsyuan commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r389829592 ## File path: core/src/test/resources/sql/winagg.iq ## @@ -431,14 +431,14 @@ join ( from "hr"."emps"

[GitHub] [calcite] vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule

2020-03-09 Thread GitBox
vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule URL: https://github.com/apache/calcite/pull/1848#discussion_r389754090 ## File path:

[GitHub] [calcite] chunweilei commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule

2020-03-09 Thread GitBox
chunweilei commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule URL: https://github.com/apache/calcite/pull/1848#discussion_r389749749 ## File path:

[GitHub] [calcite] chunweilei commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule

2020-03-09 Thread GitBox
chunweilei commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule URL: https://github.com/apache/calcite/pull/1848#discussion_r389748026 ## File path:

[GitHub] [calcite] vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule

2020-03-09 Thread GitBox
vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule URL: https://github.com/apache/calcite/pull/1848#discussion_r389726120 ## File path:

[GitHub] [calcite] vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule

2020-03-09 Thread GitBox
vlsi commented on a change in pull request #1848: [CALCITE-3845] CASE WHEN expression with nullability CAST is considered as reduced wrongly in ReduceExpressionsRule URL: https://github.com/apache/calcite/pull/1848#discussion_r389708007 ## File path:

[GitHub] [calcite] danny0405 commented on a change in pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron…

2020-03-09 Thread GitBox
danny0405 commented on a change in pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron… URL: https://github.com/apache/calcite/pull/1850#discussion_r389669351 ## File path: core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java

[GitHub] [calcite] vvysotskyi commented on a change in pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron…

2020-03-09 Thread GitBox
vvysotskyi commented on a change in pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron… URL: https://github.com/apache/calcite/pull/1850#discussion_r389624274 ## File path: core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java

[GitHub] [calcite] danny0405 opened a new pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron…

2020-03-09 Thread GitBox
danny0405 opened a new pull request #1850: [CALCITE-3847] Decorrelation for join with lateral table outputs wron… URL: https://github.com/apache/calcite/pull/1850 …g plan if the join condition contains correlation variables

[GitHub] [calcite] danny0405 commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance

2020-03-09 Thread GitBox
danny0405 commented on a change in pull request #1840: [CALCITE-3753] Remove rule queue importance URL: https://github.com/apache/calcite/pull/1840#discussion_r389569220 ## File path: core/src/test/resources/sql/winagg.iq ## @@ -431,14 +431,14 @@ join ( from