[SPARK-36822] BroadcastNestedLoopJoinExec should use all condition instead of non-equi condition

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.2.0
• Fix Version/s: 3.3.0
• Component/s: SQL
• Assignee: XiDuo You
• Reporter: XiDuo You
• Created: 22/Sep/21 01:57
• Updated: 22/Sep/21 09:26
• Resolved: 22/Sep/21 09:26

Description

At JoinSelection, with ExtractEquiJoinKeys, we use `nonEquiCond` as the join condition. It's wrong since there should exist some equi condition.

Seq(joins.BroadcastNestedLoopJoinExec(
  planLater(left), planLater(right), buildSide, joinType, nonEquiCond))

But it's should not be a bug, since we always use the smj as the default join strategy for ExtractEquiJoinKeys.

Issue Links

• links to: [Github] Pull Request #34065 (ulysses-you)
• links to: [Github] Pull Request #34065 (ulysses-you)