-
Notifications
You must be signed in to change notification settings - Fork 0
Patched DataFusion version 45.0.0
#54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
010b01c to
6762c73
Compare
1353531 to
6405b35
Compare
45.0.0
b1c4a07 to
c915be1
Compare
|
This auto-closed. |
…rceDistribution) which later causes an error during EnforceSort (without our patch). The next DataFusion version 46 upgrade does the proper fix, which is to not insert the coalesce in the first place. test: recreating the iox plan: * demonstrate the insertion of coalesce after the use of column estimates, and the removal of the test scenario's forcing of rr repartitioning test: reproducer of SanityCheck failure after EnforceSorting removes the coalesce added in the EnforceDistribution fix: special case to not remove the needed coalesce
b7d3c03 to
4f816b8
Compare
* Bump sccache version to latest to fix gh cache issue. * version blocked, trying with a hash * disable sccache.
* Bump MSRV to 1.82, toolchain to 1.85 * Fix some clippy warnings * Fix more clippy warnings
|
The security audit CI will fail due to: But the pyo3 upgraded cannot be done until the arrow upgrade (Apr 4 commit: b717723) |
|
I'm not going to debug the last few CI items, since they do not impact us. |
|
Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days. |
|
We are past this version. Closing. |
Follow on to
This brings us up to the version45 release, this Feb 3rd commit here and this apache branch.
Patches due to tech debt (all slated as lower priority, and are lingering):
chore: default=true for skip_physical_aggregate_schema_check, and add warn loggingPatches due to upstream bugs:
chore: skip order calculation / exponential planningUNIONandORDER BYqueries apache/datafusion#13748Patches because we haven't caught up yet:
fix: temporary fix to handle incorrect coalesce (inserted during EnforceDistribution) which later causes an error during EnforceSort (without our patch). The next DataFusion version 46 upgrade does the proper fix, which is to not insert the coalesce in the first place.fix(ci): build error with wasmSpecify rust toolchain explicitly, document how to change it (#14655)Bump MSRV to 1.82, toolchain to 1.85 (#14811)chore: get wasm to build in CI.Disable sccache action to fix gh cache issue (https://github.com/apache/datafusion/pull/15536)test: update sqllogictest errors.chore: fix clippy::large_enum_variant for DataFusionErrorPatches due to iox needs (unclear if upstream action TBD):
fix: handle when the left side of the union has no fields (e.g. an empty projection)count(*)where we aggregate to count rows after the projection.