| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Bump the workspace `datafusion`, `datafusion-proto`, `datafusion-spark`, and `datafusion-substrait` deps from 54.1.0 to 55.0.0, and `arrow` from 58 to 59 to match what DataFusion 55 pulls in. `object_store` stays on 0.13, which is still what DataFusion 55 depends on. Upstream changes that required code updates: - The `datafusion.proto` schema moved from `datafusion/proto/proto/` to `datafusion/proto-models/proto/`, so the `download-maven-plugin` URL and both `<sha512>` checksums are updated. - `ListingTableScanNode.target_partitions` was removed (now `reserved 9`); the partition count comes from the session config instead. - The concrete `Default*Cache` types were collapsed into a single generic `DefaultCache<K, V>`. Rather than name the new type, `cache_manager` now only sets the cache *limits* and leaves the slots `None`, which is what `CacheManager::try_new` already used to build the built-in caches on both 54 and 55 — so this is behavior-preserving. - `ExecutionPlan` gained a required `apply_expressions`; `JavaScanExec` is a leaf scan with no physical expressions, so it returns `Continue`. - `datafusion_substrait::serializer::deserialize_bytes` is now synchronous and takes `&[u8]`.
There was a problem hiding this comment.
LGTM, thanks @andygrove
Sorry, something went wrong.
|
Thanks @milenkovicm and @timsaucer |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Which issue does this PR close?
N/A — routine dependency bump; no tracking issue was filed.
Rationale for this change
Keeps the binding current with upstream DataFusion. 55.0.0 is the latest release, and staying close to it keeps the next bump small and makes upstream fixes available to Java callers.
What changes are included in this PR?
Bumps datafusion, datafusion-proto, datafusion-spark and datafusion-substrait from 54.1.0 to 55.0.0, and arrow from 58 to 59 to match what DataFusion 55 resolves to. object_store stays on 0.13 — DataFusion 55 still depends on ^0.13.2 — so the pin comment still holds.
The datafusion.version Maven property moves in lock step, since it selects the upstream tag the datafusion.proto / datafusion_common.proto definitions are downloaded from — the generated Java protobuf classes must match what datafusion-proto 55.0.0 decodes.
Adapting to the upstream API changes the bump requires:
Are these changes tested?
Covered by the existing suites — this is a dependency bump, so the value is in the current tests continuing to pass against the new version rather than in new assertions.
Are there any user-facing changes?
No API changes, and no behavior changes originating in this repo — the cache-manager rework above is deliberately behavior-preserving. Callers inherit whatever upstream changed between DataFusion 54.1.0 and 55.0.0.