Skip to content

Conversation

LiaCastaneda
Copy link

@LiaCastaneda LiaCastaneda commented Sep 2, 2025

Which issue does this PR close?

Cherry picks all Hash Join Dynamic filtering work done until now, not availabe on V49

adriangb and others added 6 commits September 2, 2025 16:14
* Add ExecutionPlan::reset_state

Co-authored-by: Robert Ream <[email protected]>

* Update datafusion/sqllogictest/test_files/cte.slt

* Add reference

* fmt

* add to upgrade guide

* add explain plan, implement in more plans

* fmt

* only explain

---------

Co-authored-by: Robert Ream <[email protected]>
* dissallow pushdown of volatile PhysicalExprs

* fix

* add FilteredVec helper to handle filter / remap pattern (#34)

* checkpoint: Address PR feedback in https://github.com/apach...

* add FilteredVec to consolidate handling of filter / remap pattern

* lint

* Add slt test for pushing volatile predicates down (#35)

---------

Co-authored-by: Andrew Lamb <[email protected]>
(cherry picked from commit 94e8548)
@LiaCastaneda LiaCastaneda marked this pull request as ready for review September 8, 2025 07:10
Copy link

@gabotechs gabotechs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the MSRV failing pipelines are worth a look, I think they should be passing. Otherwise, LGTM

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Sep 9, 2025

View all feedbacks in Devflow UI.

2025-09-09 14:05:00 UTC ℹ️ Start processing command /merge


2025-09-09 14:05:13 UTC ❌ MergeQueue

This PR is already merged

@LiaCastaneda LiaCastaneda merged commit 013d4ad into branch-49 Sep 9, 2025
56 of 59 checks passed
@LiaCastaneda LiaCastaneda changed the title Cherry pick dynamic filtering work [branch-49] Cherry pick dynamic filtering work Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants