Skip to content

[dynamic filtering] remove extra proto roundtrips in apply_dynamic_filter_update #768

New issue
New issue
Open
Open
[dynamic filtering] remove extra proto roundtrips in apply_dynamic_filter_update#768
Labels
df56This issue is blocked by the DataFusion 56 upgrade

Description

@jayshrivastava
jayshrivastava
opened on Oct 7, 2026

See

datafusion-distributed/src/dynamic_filtering/mod.rs

Line 234 in 0b3a283

pub(crate) fn apply_dynamic_filter_update(
.

We should avoid the proto conversion here, possibly by making the unmapped expression / original children public in upstream datafusion.

Activity

  1. jayshrivastava
    added a parent issue on Oct 7, 2026
  2. jayshrivastava
    added
    df56This issue is blocked by the DataFusion 56 upgrade
    on Oct 21, 2026
  3. dwsmith1983
    added a commit that references this issue on Oct 27, 2026

    datafusion-physical-expr: add `DynamicFilterPhysicalExpr::update_from…

    914fc1b

  4. jayshrivastava commented on Nov 4, 2026

    @jayshrivastava
    jayshrivastava
    on Nov 4, 2026
    CollaboratorAuthor

    This can be done after df56 due to apache/datafusion#26534

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    df56This issue is blocked by the DataFusion 56 upgrade

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions