alamb opened a new pull request, #10398: URL: https://github.com/apache/arrow-rs/pull/10398
# Which issue does this PR close? - Part of #10308 # Rationale for this change #10354 proposes removing the `object_store` integration from the `parquet` crate to slim down the dependency chain. Removing `ParquetObjectReader` / `ParquetObjectWriter` outright would be a jarring upgrade, so per [this discussion](https://github.com/apache/arrow-rs/pull/10354#pullrequestreview-4738249835) we plan to do it in two steps: 1. Deprecate `ParquetObjectReader` and `ParquetObjectWriter`, and point users at an example showing how to inline the same code in downstream crates (this PR) 2. Actually remove them (a future PR) # What changes are included in this PR? 1. Deprecate `ParquetObjectReader` and `ParquetObjectWriter` as of version 60.0.0, with notes pointing at the new example 2. Add `parquet/examples/object_store.rs`, a runnable, self-contained copy of the reader/writer implementations (and their tests) that users can copy/paste into their own crates 3. Add `#[allow(deprecated)]` to internal uses (tests / benches); internal code will be migrated when the structs are removed # Are these changes tested? Yes: the example includes the existing unit tests for the reader and writer, and runs as `cargo run --example object_store --features arrow,async` # Are there any user-facing changes? Yes: `ParquetObjectReader` and `ParquetObjectWriter` are now marked deprecated. No functionality is removed yet. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
