jhorstmann commented on PR #9778: URL: https://github.com/apache/arrow-rs/pull/9778#issuecomment-4318627563
I share @alamb's concern that in some cases this might now require a copy, where it did not need one before, and these cases are probably not covered by the benchmark. We now rely on the alignment of the global allocator, which could be changed by a user, and which might also depend on the size of the allocation. For smaller ipc message blocks, the allocator might use a smaller alignment. I don't see a good way now to avoid these copies, apart from waiting for improvements in the rust standard library itself (https://github.com/rust-lang/rfcs/blob/master/text/2930-read-buf.md#summary). -- 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]
