lidavidm commented on PR #45093: URL: https://github.com/apache/arrow/pull/45093#issuecomment-2562033172
> This patch LGTM now. Out of curiousity, why not call `PreBuffer` in `SerializedFile`, but call this new interface? The idea is that this doesn't actually trigger any I/O. I suppose you could rig a custom RandomAccessFile to do the same thing with PreBuffer but I think it would be significantly more complicated (as you do want to allow I/O to read the footer, and you'll have to resolve the I/O request at some point). -- 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]
