[ 
https://issues.apache.org/jira/browse/CALCITE-4759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17409180#comment-17409180
 ] 

Julian Hyde commented on CALCITE-4759:
--------------------------------------

Good to know.

The proximate cause for this problem is that there is no test for either 
instance of this rule in {{RelOptRulesTest}}. First step to solving this 
problem would be to add a couple of tests.

> 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 
> 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



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

Reply via email to