[jira] [Commented] (CALCITE-3169) decorrelateRel method should return when meeting SEMI/ANTI join in RelDecorrelator

["Julian Hyde (JIRA)" <ji...@apache.org>]

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

Julian Hyde commented on CALCITE-3169:
--------------------------------------

Can you change the description to something meaningful to the end-user? You've described the code change, not the problem.

> decorrelateRel method should return when meeting SEMI/ANTI join in RelDecorrelator
> ----------------------------------------------------------------------------------
>
>                 Key: CALCITE-3169
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3169
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.20.0
>            Reporter: godfrey he
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.21.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> {code:java}
> /**
>    * Rewrite LogicalJoin.
>    *
>    * @param rel Join
>    */
>   public Frame decorrelateRel(LogicalJoin rel) {
>     // For SEMI/ANTI join decorrelate it's input directly,
>     // because the correlate variables can only be propagated from
>     // the left side, which is not supported yet.
>     if (!rel.getJoinType().projectsRight()) {
>       decorrelateRel((RelNode) rel);
>     }
> ...
> {code}
> {{return}} is missing before {{decorrelateRel((RelNode) rel);}}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)