You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2026/10/01 08:41:26 UTC
Re: [PR] Make StatisticsContext cache entries retain plan nodes [datafusion]
github-actions[bot] commented on PR #25929:
URL: https://github.com/apache/datafusion/pull/25929#issuecomment-5927876954
<!-- semver-check-comment -->
Thank you for opening this pull request!
Reviewer note: [cargo-semver-checks](https://github.com/obi1kenobi/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>
<summary>Details</summary>
```
Cloning apache/main
Building datafusion-physical-plan v55.1.0 (current)
Built [ 39.214s] (current)
Parsing datafusion-physical-plan v55.1.0 (current)
Parsed [ 0.151s] (current)
Building datafusion-physical-plan v55.1.0 (baseline)
Built [ 37.588s] (baseline)
Parsing datafusion-physical-plan v55.1.0 (baseline)
Parsed [ 0.150s] (baseline)
Checking datafusion-physical-plan v55.1.0 -> v55.1.0 (no change; assume patch)
Checked [ 0.843s] 223 checks: 222 pass, 1 fail, 0 warn, 31 skip
--- failure method_requires_different_generic_type_params: method now requires a different number of generic type parameters ---
Description:
A method now requires a different number of generic type parameters than it used to. Uses of this method that supplied the previous number of generic types will be broken.
ref: https://doc.rust-lang.org/reference/items/generics.html
impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/method_requires_different_generic_type_params.ron
Failed in:
datafusion_physical_plan::statistics::StatisticsContext::compute takes 1 generic types instead of 0, in /home/runner/work/datafusion/datafusion/datafusion/physical-plan/src/statistics.rs:278
datafusion_physical_plan::statistics::StatisticsContext::compute_extended takes 1 generic types instead of 0, in /home/runner/work/datafusion/datafusion/datafusion/physical-plan/src/statistics.rs:297
datafusion_physical_plan::StatisticsContext::compute takes 1 generic types instead of 0, in /home/runner/work/datafusion/datafusion/datafusion/physical-plan/src/statistics.rs:278
datafusion_physical_plan::StatisticsContext::compute_extended takes 1 generic types instead of 0, in /home/runner/work/datafusion/datafusion/datafusion/physical-plan/src/statistics.rs:297
Summary semver requires new major version: 1 major and 0 minor checks failed
Finished [ 79.889s] datafusion-physical-plan
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org