chore: Update to arrow/parquet 59.0.0#22744
Conversation
| .bloom_filter_properties(&ColumnPath::from("")) | ||
| .expect("expected bloom properties!") | ||
| .fpp, | ||
| .fpp(), |
There was a problem hiding this comment.
These fields are made private in
| }; | ||
| if let Some(bloom_filter_ndv) = bloom_filter_ndv { | ||
| builder = builder.set_bloom_filter_ndv(*bloom_filter_ndv); | ||
| builder = builder.set_bloom_filter_max_ndv(*bloom_filter_ndv); |
There was a problem hiding this comment.
- due to feat(parquet): add BloomFilterPropertiesBuilder arrow-rs#9877 which deprecated set_bloom_filtr_ndv
| ndv: DEFAULT_BLOOM_FILTER_NDV | ||
| }), | ||
| Some( | ||
| &BloomFilterProperties::builder() |
There was a problem hiding this comment.
Properties are now built with builder:
| &[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0], | ||
| &[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 0, 1, 2, 3, 5, 6], | ||
| ]))], | ||
| vec![Arc::new( |
There was a problem hiding this comment.
from impl was removed because it could panic
| scale: 2, | ||
| precision: 9, | ||
| }) | ||
| .with_logical_type(LogicalType::decimal(2, 9)) |
There was a problem hiding this comment.
have to use new helpers added in
| ] | ||
|
|
||
| [[package]] | ||
| name = "integer-encoding" |
There was a problem hiding this comment.
yay for removing older deps
| "cfg-if", | ||
| ] | ||
|
|
||
| [[package]] |
There was a problem hiding this comment.
no more thrift! We now use the entirely new thrift encoder and not the thrift generator
|
Thank you for opening this pull request! Reviewer note: 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 |
Which issue does this PR close?
59.0.0(May 2026) arrow-rs#9110WIP: I am using this PR to test the arrow release
Rationale for this change
Update to latest version of arrow/parquet
What changes are included in this PR?
Are these changes tested?
By CI
Are there any user-facing changes?
New dependency