Skip to content

minor: add Any to QueryPlanner trait#22241

Open
milenkovicm wants to merge 1 commit into
apache:mainfrom
milenkovicm:minor_query_planner_any
Open

minor: add Any to QueryPlanner trait#22241
milenkovicm wants to merge 1 commit into
apache:mainfrom
milenkovicm:minor_query_planner_any

Conversation

@milenkovicm
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

  • does not close the issue.

Rationale for this change

Working on #22151 it seams that QueryPlanner should extend Any in order to be downcastad to actual implementation. The main reason behind cast is to support runtime change of inner query planner.

What changes are included in this PR?

  • trait QueryPlanner extends Any

Are these changes tested?

Using existing tests

Are there any user-facing changes?

this would be backward incompatible change

@github-actions github-actions Bot added the core Core DataFusion crate label May 16, 2026
@github-actions
Copy link
Copy Markdown

Thank you for opening this pull request!

Reviewer note: cargo-semver-checks reported the current version number is not SemVer-compatible with the changes in this pull request (compared against the base branch).

Details
     Cloning apache/main
    Building datafusion v53.1.0 (current)
       Built [  99.190s] (current)
     Parsing datafusion v53.1.0 (current)
      Parsed [   0.038s] (current)
    Building datafusion v53.1.0 (baseline)
       Built [  95.850s] (baseline)
     Parsing datafusion v53.1.0 (baseline)
      Parsed [   0.037s] (baseline)
    Checking datafusion v53.1.0 -> v53.1.0 (no change; assume patch)
     Checked [   0.811s] 222 checks: 221 pass, 1 fail, 0 warn, 30 skip

--- failure trait_added_supertrait: non-sealed trait added new supertraits ---

Description:
A non-sealed trait added one or more supertraits, which breaks downstream implementations of the trait
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#generic-bounds-tighten
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.47.0/src/lints/trait_added_supertrait.ron

Failed in:
  trait datafusion::execution::context::QueryPlanner gained Any in file /home/runner/work/datafusion/datafusion/datafusion/core/src/execution/context/mod.rs:2077

     Summary semver requires new major version: 1 major and 0 minor checks failed
    Finished [ 198.837s] datafusion

@github-actions github-actions Bot added the auto detected api change Auto detected API change label May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto detected api change Auto detected API change core Core DataFusion crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant