friendlymatthew opened a new issue, #9197: URL: https://github.com/apache/arrow-rs/issues/9197
https://github.com/apache/arrow-rs/pull/8071 replaced the previous boolean-based page index configuration with the `PageIndexPolicy` enum, expanding the behavior from 2 states to 3. However, the original boolean setter on `ArrowReaderOptions` was not deprecated, leaving 2 overlapping apis that configure the same behavior. This duplication (and not deprecating) probably increases the likelihood of subtle misconfiguration, especially as additional states are introduced without being surfaced to users See: https://github.com/apache/arrow-rs/blob/1035781d68e384bb305295c38175b4adabc419aa/parquet/src/arrow/arrow_reader/mod.rs#L604-L633 We _do_ deprecate a similar function in the `ParquetMetadataReader`: https://github.com/apache/arrow-rs/blob/1035781d68e384bb305295c38175b4adabc419aa/parquet/src/file/metadata/reader.rs#L120-L153 -- 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]
