[CALCITE-3169] RelDecorrelator should return for SEMI/ANTI join decorrelate

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.20.0
• Fix Version/s: 1.21.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Danny Chen
• Reporter: godfrey he
• Created: 03/Jul/19 04:33
• Updated: 10/Sep/19 21:37
• Resolved: 08/Jul/19 13:05

Description

/**
   * 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);
    }
...

return is missing before decorrelateRel((RelNode) rel);

Issue Links

• Is contained by: CALCITE-3170 ANTI join on conditions push down generates wrong plan
• links to: GitHub Pull Request #1296