Skip to content

[branch-55] Fix panic on RightMark hash joins when propagating ordering (#24718) - #24759

Merged
timsaucer merged 1 commit into
apache:branch-55from
timsaucer:backport/pr-24718-branch-55
Aug 29, 2026
Merged

[branch-55] Fix panic on RightMark hash joins when propagating ordering (#24718)#24759
timsaucer merged 1 commit into
apache:branch-55from
timsaucer:backport/pr-24718-branch-55

Conversation

@timsaucer

Copy link
Copy Markdown
Member

This PR is a backport of #24718 onto branch-55 for the purpose of a patch release.

…4718)

## Which issue does this PR close?

- Closes apache#24717 (affecting DF 55)

## Rationale for this change

Fixes a panic in `enforce_sorting` when given a `RightMark` hash join:
`HashjoinExec::maintains_input_order` has been changed to also return
`true` on `RightMark` but `enforce_sorting` has not been updated

## What changes are included in this PR?

Adds support of `RightMark` joins to `enforce_sorting` they keep the
input sorting.
@github-actions github-actions Bot added optimizer Optimizer rules core Core DataFusion crate labels Aug 28, 2026
@timsaucer timsaucer changed the title Fix panic on RightMark hash joins when propagating ordering (#24718) [branch-55] Fix panic on RightMark hash joins when propagating ordering (#24718) Aug 28, 2026
@timsaucer

Copy link
Copy Markdown
Member Author

@Tpt Would you mind reviewing as the original author? It was a trivial cherry pick.

@timsaucer
timsaucer marked this pull request as ready for review August 28, 2026 14:45
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.17%. Comparing base (1a944f4) to head (16e9cae).

Additional details and impacted files
@@              Coverage Diff              @@
##           branch-55   #24759      +/-   ##
=============================================
- Coverage      81.17%   81.17%   -0.01%     
=============================================
  Files           1110     1110              
  Lines         387043   387043              
  Branches      387043   387043              
=============================================
- Hits          314186   314177       -9     
- Misses         54366    54373       +7     
- Partials       18491    18493       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@timsaucer
timsaucer merged commit 1540853 into apache:branch-55 Aug 29, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core DataFusion crate optimizer Optimizer rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants