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

   # Which issue does this PR close?
   
   - Part of https://github.com/apache/arrow-rs/issues/8000
   - Prep PR for https://github.com/apache/arrow-rs/pull/8340, to make it 
easier to review
   
   # Rationale for this change
   
   In https://github.com/apache/arrow-rs/pull/8340 I am trying to split the 
"IO" from the "where is the metadata in the file" from the "decode thrift into 
Rust structures" logic. 
   
   I want to make it as easy as possible to review so I split it into pieces, 
but you can see https://github.com/apache/arrow-rs/pull/8340 for how it all 
fits together
   
   # What changes are included in this PR?
   
   This PR cleans up the code that handles parsing the 8 byte parquet file 
footer, `FooterTail`, into its own module and construtor
   
   # Are these changes tested?
   
   yes, by CI
   
   
   # Are there any user-facing changes?
   
   No, this is entirely internal reorganization


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