pitrou commented on issue #33933: URL: https://github.com/apache/arrow/issues/33933#issuecomment-1410037938
> No checks here https://github.com/apache/arrow/blob/master/cpp/src/arrow/buffer.h#L91 That's because 64-byte alignment is recommended but not mandatory. Arrow C++ must be able to access memory allocated or provided by other libraries, which will not be 64-byte aligned (or perhaps not aligned at all as @westonpace mentions). > Could be fixed with: Why not, but should be `alignas(kDefaultBufferAlignment)` then. -- 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]
