alamb commented on PR #9778: URL: https://github.com/apache/arrow-rs/pull/9778#issuecomment-4317000922
> Yes @alamb , that’s a fair point. > > The only new copy introduced by this fix should be limited to dense union offsets, and only when the Vec-backed body buffer is not aligned for `i32`. So the tradeoff is: I think it applies more broadly than this -- basically for any type of buffer that requires alignment For example, a UInt64Array can now require another copy (if the Vec wasn't 64 bit aligned). However, this copy happens sliently today because the array builder sorts it out, right? -- 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]
