Skip to content

Remove the statistics() api in execution plan#20319

Merged
xudong963 merged 2 commits intoapache:mainfrom
xudong963:remove_stats_api
Feb 13, 2026
Merged

Remove the statistics() api in execution plan#20319
xudong963 merged 2 commits intoapache:mainfrom
xudong963:remove_stats_api

Conversation

@xudong963
Copy link
Copy Markdown
Member

The API has been marked as deprecated for about 10 months, and we're planning to do more changes for statistics. I think it's time to let it go.

@github-actions github-actions Bot added optimizer Optimizer rules core Core DataFusion crate datasource Changes to the datasource crate ffi Changes to the ffi crate physical-plan Changes to the physical-plan crate labels Feb 12, 2026
Copy link
Copy Markdown
Contributor

@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.

👍 nice cleanup

Copy link
Copy Markdown
Contributor

@jonathanc-n jonathanc-n left a comment

Choose a reason for hiding this comment

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

Thanks nice job!

@xudong963 xudong963 changed the title Remove the statistics in execution plan Remove the statistics() api in execution plan Feb 12, 2026
@alamb alamb added the api change Changes the API exposed to users of the crate label Feb 12, 2026
@xudong963 xudong963 added this pull request to the merge queue Feb 13, 2026
Merged via the queue into apache:main with commit f48dc72 Feb 13, 2026
33 checks passed
de-bgunter pushed a commit to de-bgunter/datafusion that referenced this pull request Mar 24, 2026
The API has been marked as deprecated for about 10 months, and we're
planning to do more
[changes](apache#20184) for
statistics. I think it's time to let it go.
github-merge-queue Bot pushed a commit that referenced this pull request Apr 9, 2026
## Which issue does this PR close?

- Related to #19692

## Rationale for this change

@rluvaton noted some issues with the 53 upgrade guide
#19692 (comment):

> FYI, the migration guide says 53.0.0 was not released yet and it miss
the following breaking changes:
> 
> #20319 - removed deprecated
function from ExecutionPlan trait
> #19635 - added argument to
HashJoinExec::try_new function

## What changes are included in this PR?

Fix the above

## Are these changes tested?
Yes by CI

## Are there any user-facing changes?

Better docs
Rich-T-kid pushed a commit to Rich-T-kid/datafusion that referenced this pull request Apr 21, 2026
## Which issue does this PR close?

- Related to apache#19692

## Rationale for this change

@rluvaton noted some issues with the 53 upgrade guide
apache#19692 (comment):

> FYI, the migration guide says 53.0.0 was not released yet and it miss
the following breaking changes:
> 
> apache#20319 - removed deprecated
function from ExecutionPlan trait
> apache#19635 - added argument to
HashJoinExec::try_new function

## What changes are included in this PR?

Fix the above

## Are these changes tested?
Yes by CI

## Are there any user-facing changes?

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

Labels

api change Changes the API exposed to users of the crate core Core DataFusion crate datasource Changes to the datasource crate ffi Changes to the ffi crate optimizer Optimizer rules physical-plan Changes to the physical-plan crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants