mbutrovich commented on PR #3670: URL: https://github.com/apache/datafusion-comet/pull/3670#issuecomment-4047059762
> I was just looking into this PR. A note: ptr::read()/read_unaligned() read in native order. Spark Unsafe uses Little Endian. So, the new code won't work on Big Endian machines. AFAIK arrow-rs' position on big endian support has not changed (i.e., it does not support big endian). https://github.com/apache/arrow-rs/issues/6917#issuecomment-2564302613 It's possible we've acquired some code that supports both targets, but we likely only need to consider little endian. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
