alamb opened a new pull request, #10825:
URL: https://github.com/apache/arrow-rs/pull/10825

   # Which issue does this PR close?
   
   - Closes #10824.
   
   # Rationale for this change
   
   #10719 introduced the `PageIndex` struct but its constructor is 
`pub(crate)`, so external code can no longer assemble `ColumnIndexMetaData` / 
`OffsetIndexMetaData` values (whose builders are still public) into a 
`ParquetMetaData`. DataFusion relies on this to construct synthetic metadata in 
tests that exercise page-level statistics handling (see 
apache/datafusion#24366).
   
   # What changes are included in this PR?
   
   - Make `PageIndex::new` `pub` and document its arguments
   - Add a doc example showing how to construct a synthetic `PageIndex` and 
attach it to a `ParquetMetaData` via `ParquetMetaDataBuilder::set_page_index`, 
modeled on the DataFusion test code that needs this API
   
   # Are these changes tested?
   
   Yes, by the new doc example.
   
   # Are there any user-facing changes?
   
   `PageIndex::new` is now part of the public API. No breaking changes.


-- 
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]

Reply via email to