EnricoMi commented on PR #44279: URL: https://github.com/apache/arrow/pull/44279#issuecomment-2705672729
@lidavidm this change uses `util::EnsureAlignment`, which errors for non-CPU buffers: https://github.com/apache/arrow/pull/44279/files#diff-fbfb27689f69623e457521d861b3a39557fa99394b47c57e4143da5b1aa575bfR162 This will break existing code. It would require users to set FlightCallOptions.read_options.ensure_memory_alignment = False Should this re-alignment silently be skipped for non-CPU buffers? This would be the best place to skip re-alignment, but we the batch at this point, not individual buffers: https://github.com/apache/arrow/pull/44279/files#diff-e992169684aea9845ac776ada4cbb2b5dc711b49e5a3fbc6046c92299e1aefceR642-R645 -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
