ranflarion commented on PR #10522: URL: https://github.com/apache/arrow-rs/pull/10522#issuecomment-5168917898
> * looks like it supersedes [fix(arrow-ipc): bound MessageReader allocations by actual stream bytes #9869](https://github.com/apache/arrow-rs/pull/9869) yes i missed that, #9869 is after the same three changes (take/read_to_end for the metadata, try_from on bodyLength, bounded body reads). The difference is the body path: #9869 copies through a 64KiB scratch chunk via extend_from_slice, so every body pays a second full copy, while here the bytes land directly in the destination MutableBuffer. -- 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]
