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

   # Which issue does this PR close?
   
   - Part of #5853
   - Closes #8517.
   
   # Rationale for this change
   
   A good bit (around 15%) of Parquet metadata parsing involves first decoding 
to thrift structs (`FileMetaData`, `RowGroup`, etc), and then converting to the 
metadata structs used by this crate (`ParquetMetaData`, `RowGroupMetaData`, 
etc). This PR removes some of the intermediate structures and parses straight 
to the crate structs.
   
   # What changes are included in this PR?
   
   Some thrift generated structures are removed, and the code necessary to 
decode has been hand written. This will enable future optimizations such as 
selectively decoding parts of the metadata.
   
   In addition to the above, this PR cleans up some of the memory size 
computation, and also boxes some of the objects used for decryption.
   
   # Are these changes tested?
   
   Should be covered by existing tests.
   
   # Are there any user-facing changes?
   
   No, only private APIs are changed.
   


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