felipecrv commented on issue #2526: URL: https://github.com/apache/arrow-adbc/issues/2526#issuecomment-2660680529
> FWIW, the allocator in the C# implementation of Arrow will overallocate by 64 bytes and then adjust the starting location to be on a 64-byte boundary. Which is great. That means that when `ArrayData::align_buffers()` is called on the Rust side, no buffer needs to be copied to be correctly aligned. I think the final stance on this topic should be that: 1. producers must try their best to produce aligned buffers 2. Rust consumers should always validate the alignment because they shouldn't trust external data and the risk of a panic is not worth the time saved skipping the alignment check (which will rarely trigger a copy of any buffers). -- 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]
