Uh oh!
There was an error while loading. Please reload this page.
Upgrade DataFusion to arrow-rs/parquet 57.2.0 - #19355
Conversation
7ec33db to
24d3a15Compare| # Upstream arrow-rs issue: https://github.com/apache/arrow-rs/issues/8841 | ||
| # This should succeed after we receive the fix | ||
| query error Arrow error: Compute error: Internal Error: Cannot cast BinaryView to BinaryArray of expected type | ||
| query I |
There was a problem hiding this comment.
I think this fix will also close #19290; quickly tested checking this PR branch out and trying the test in the issue and it seems to execute successfully
There was a problem hiding this comment.
I updated the PR description to close #19290 as well
| # coerce structs with different field orders, | ||
| # (note the *value*s are from column2 but the field name is 'xxx', as the coerced | ||
| # type takes the field name from the last argument (column3) | ||
| # should keep the same field values |
| drop table t; | ||
| query error DataFusion error: Optimizer rule 'simplify_expressions' failed[\s\S]*Arrow error: Cast error: Cannot cast string 'a' to value of Float64 type | ||
| statement ok |
Dandandan
commented
Dec 21, 2025
run benchmarks |
Jefffrey
commented
Dec 21, 2025
FYI @tobixdev can keep an eye on this PR as it should include the fix from arrow-rs needed to do #19085 -> can start it once this PR merges |
alamb
commented
Dec 23, 2025
run benchmarks |
This comment was marked as outdated.
This comment was marked as outdated.
alamb-ghbot
commented
Dec 23, 2025
🤖: Benchmark completed Details |
alamb
commented
Dec 24, 2025
run benchmarks |
This comment was marked as outdated.
This comment was marked as outdated.
alamb-ghbot
commented
Dec 24, 2025
🤖: Benchmark completed Details |
24d3a15 to
ff98f2fComparealamb
commented
Jan 6, 2026
run benchmarks |
4155056 to
70e2841Comparealamb-ghbot
commented
Jan 6, 2026
🤖 |
alamb-ghbot
commented
Jan 7, 2026
🤖: Benchmark completed Details |
alamb
commented
Jan 7, 2026
run benchmark clickbench_partitioned |
alamb-ghbot
commented
Jan 7, 2026
🤖 |
ca11966 to
4cf6e96Compare| ScalarValue::try_new_null(&DataType::Map(map_field_ref, false)).unwrap(), | ||
| ScalarValue::try_new_null(&DataType::Union( | ||
| UnionFields::new(vec![42], vec![field_ref]), | ||
| UnionFields::try_new(vec![42], vec![field_ref]).unwrap(), |
There was a problem hiding this comment.
Changes due to this one from @friendlymatthew
| # Upstream arrow-rs issue: https://github.com/apache/arrow-rs/issues/8841 | ||
| # This should succeed after we receive the fix | ||
| query error Arrow error: Compute error: Internal Error: Cannot cast BinaryView to BinaryArray of expected type | ||
| query I |
There was a problem hiding this comment.
I updated the PR description to close #19290 as well
| let type_ids = 0_i8..fields.len() as i8; | ||
| DataType::Union(UnionFields::new(type_ids, fields), UnionMode::Dense) | ||
| DataType::Union( | ||
| UnionFields::try_new(type_ids, fields).unwrap(), |
There was a problem hiding this comment.
Should we propagate this unwrap?
There was a problem hiding this comment.
Since the type_ids are autoincrementing, we can probably use UnionFields::from_fields(fields) and avoid the Result
UnionFields::from_fields is a convenience constructor that will infallibly construct the UnionFields (since the only check we really need is to ensure type id uniqueness). https://docs.rs/arrow/latest/arrow/datatypes/struct.UnionFields.html#method.from_fields
| let type_ids = 0_i8..fields.len() as i8; | ||
| DataType::Union(UnionFields::new(type_ids, fields), UnionMode::Dense) | ||
| DataType::Union( | ||
| UnionFields::try_new(type_ids, fields).unwrap(), |
There was a problem hiding this comment.
Since the type_ids are autoincrementing, we can probably use UnionFields::from_fields(fields) and avoid the Result
UnionFields::from_fields is a convenience constructor that will infallibly construct the UnionFields (since the only check we really need is to ensure type id uniqueness). https://docs.rs/arrow/latest/arrow/datatypes/struct.UnionFields.html#method.from_fields
alamb
commented
Jan 13, 2026
Let's do this! |
Uh oh!
There was an error while loading. Please reload this page.
…s methods (#19797) These PRs are available to us now as part of upgrade to arrow-s 57.2.0 (#19355): - apache/arrow-rs#8993 - apache/arrow-rs#9040 Make use of them in some refactorings here.
…#19728) ## Which issue does this PR close? - Follow on to #19355 - related to apache/arrow-rs#8466 - Closes#17455 ## Rationale for this change Keep datafusion up to date (and test Arrow using DataFusion tests) ## What changes are included in this PR? <!-- There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR. --> ## Are these changes tested? <!-- We typically require tests for all PRs in order to: 1. Prevent the code from being accidentally broken by subsequent changes 2. Serve as another way to document the expected behavior of the code If tests are not included in your PR, please explain why (for example, are they covered by existing tests)? --> ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> --------- Co-authored-by: Daniël Heres <danielheres@gmail.com>
Includes fix for FixedSizeBinary LEFT JOIN bug - apache/arrow-rs#8981 Cherry-picked test and API updates from - apache#19355
Includes fix for FixedSizeBinary LEFT JOIN bug - apache/arrow-rs#8981 Cherry-picked test and API updates from - apache#19355
Includes fix for FixedSizeBinary LEFT JOIN bug - apache/arrow-rs#8981 Cherry-picked test and API updates from - apache#19355
## Which issue does this PR close? - Related to apache/arrow-rs#8465 - Closesapache#19290 ## Rationale for this change Upgrade to latest arrow version I made this PR early to test the arrow release with DataFusion ## What changes are included in this PR? 1. Update arrow 2. Updates for API ## Are these changes tested? Yes by CI ## Are there any user-facing changes? No
…s methods (apache#19797) These PRs are available to us now as part of upgrade to arrow-s 57.2.0 (apache#19355): - apache/arrow-rs#8993 - apache/arrow-rs#9040 Make use of them in some refactorings here.
…apache#19728) ## Which issue does this PR close? - Follow on to apache#19355 - related to apache/arrow-rs#8466 - Closesapache#17455 ## Rationale for this change Keep datafusion up to date (and test Arrow using DataFusion tests) ## What changes are included in this PR? <!-- There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR. --> ## Are these changes tested? <!-- We typically require tests for all PRs in order to: 1. Prevent the code from being accidentally broken by subsequent changes 2. Serve as another way to document the expected behavior of the code If tests are not included in your PR, please explain why (for example, are they covered by existing tests)? --> ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> --------- Co-authored-by: Daniël Heres <danielheres@gmail.com>
Which issue does this PR close?
57.2.0(December 2025) arrow-rs#8465Dictionary(UInt8, LargeUtf8)toUtf8View#19290Rationale for this change
Upgrade to latest arrow version
I made this PR early to test the arrow release with DataFusion
What changes are included in this PR?
Are these changes tested?
Yes by CI
Are there any user-facing changes?
No