https://github.com/apache/spark
Revision 3e28f338fad66393b6d2f7a2da6ce5eee60a626e authored by ulysses-you on 05 July 2022, 03:31:02 UTC, committed by Wenchen Fan on 05 July 2022, 03:31:35 UTC
### What changes were proposed in this pull request?

Change `currentPhysicalPlan` to `inputPlan ` when we restore the broadcast exchange for DPP.

### Why are the changes needed?

The currentPhysicalPlan can be wrapped with broadcast query stage so it is not safe to match it. For example:
 The broadcast exchange which is added by DPP is running before than the normal broadcast exchange(e.g. introduced by join).

### Does this PR introduce _any_ user-facing change?

yes bug fix

### How was this patch tested?

add test

Closes #36974 from ulysses-you/inputplan.

Authored-by: ulysses-you <ulyssesyou18@gmail.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit c320a5d51b2c8427fc5d6648984bfd266891b451)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
1 parent 463a24d
History
Tip revision: 3e28f338fad66393b6d2f7a2da6ce5eee60a626e authored by ulysses-you on 05 July 2022, 03:31:02 UTC
[SPARK-39447][SQL] Avoid AssertionError in AdaptiveSparkPlanExec.doExecuteBroadcast
Tip revision: 3e28f33

README.md

back to top