apache / datafusion

Apache DataFusion SQL Query Engine
https://datafusion.apache.org/
Apache License 2.0
6.33k stars 1.2k forks source link

Fix invalid swap for LeftMark nested loops join #13426

Closed findepi closed 5 days ago

findepi commented 1 week ago

Which issue does this PR close?

Fixes https://github.com/apache/datafusion/issues/13425

Rationale for this change

Fix query regression in DF 43

What changes are included in this PR?

Proper guard for swapping potentially unswappabe join

Are these changes tested?

yes

Are there any user-facing changes?

yes

findepi commented 1 week ago

cc @eejbyfeldt @alamb

alamb commented 5 days ago

Thanks @findepi and @Dandandan