[ 
https://issues.apache.org/jira/browse/CALCITE-4759?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Julian Hyde updated CALCITE-4759:
---------------------------------
    Description: 
JoinPushThroughJoinRule doesn't ever get to the onMatch() condition, even when 
presented with a plan with two back to back joins. I found this as a test 
failure while upgrading Beam from Calcite 1.20 to 1.25.

Looks like this was introduced by [PR 
2024|https://github.com/apache/calcite/pull/2024] (CALCITE-3923), I was able to 
workaround the issue by patching in the match rule prior to this change: commit 
[6c5c04, 'Workaround CALCITE-4759 in 
JoinPushThroughJoinRule'|https://github.com/apache/beam/pull/14729/commits/6c5c04b8fc903fc0130ca06d4d8f2dbd082a3341]
 in [Beam PR 14729|https://github.com/apache/beam/pull/14729].

  was:
JoinPushThroughJoinRule doesn't ever get to the onMatch() condition, even when 
presented with a plan with two back to back joins. I found this as a test 
failure while upgrading Beam from Calcite 1.20 to 1.25.

Looks like this was introduced by https://github.com/apache/calcite/pull/2024, 
I was able to workaround the issue by patching in the match rule prior to this 
change: 
https://github.com/apache/beam/pull/14729/commits/22994a0c093c91f3af58fc4a7824e8971a449907


> JoinPushThroughJoinRule does not match two joins
> ------------------------------------------------
>
>                 Key: CALCITE-4759
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4759
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.25.0
>            Reporter: Andrew Pilloud
>            Priority: Major
>
> JoinPushThroughJoinRule doesn't ever get to the onMatch() condition, even 
> when presented with a plan with two back to back joins. I found this as a 
> test failure while upgrading Beam from Calcite 1.20 to 1.25.
> Looks like this was introduced by [PR 
> 2024|https://github.com/apache/calcite/pull/2024] (CALCITE-3923), I was able 
> to workaround the issue by patching in the match rule prior to this change: 
> commit [6c5c04, 'Workaround CALCITE-4759 in 
> JoinPushThroughJoinRule'|https://github.com/apache/beam/pull/14729/commits/6c5c04b8fc903fc0130ca06d4d8f2dbd082a3341]
>  in [Beam PR 14729|https://github.com/apache/beam/pull/14729].



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to