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

["godfrey he (JIRA)" <ji...@apache.org>]

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

godfrey he updated CALCITE-3169:
--------------------------------
    Description: 
{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);}}


  was:

{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);`



> 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
>             Fix For: 1.21.0
>
>
> {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)