etseidl commented on PR #7371: URL: https://github.com/apache/arrow-rs/pull/7371#issuecomment-2786624982
> > In my opinion, supporting metadata larger than 4GB, while admirable from a completeness point of view, is likely not a super important usecase so I think keeping the API churn down is probably a good idea. > > I think @etseidl 's point here is that since the metadata length has to fit in the last 4 bytes of the file, the Parquet spec doesn't support metadata larger than u32 anyways. Yes. But it does add some unnecessary thrash since that field is only ever initialized via a 4 byte array. No harm in leaving it as `usize`. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
