k8ika0s commented on issue #48213: URL: https://github.com/apache/arrow/issues/48213#issuecomment-3568309154
Thanks, @kou — that makes sense. The big diff in #48212 is mostly because I pulled in fresh upstream commits before adding my endianness work, so it looks a lot noisier than it really is. But I totally get that reviewing a giant PR like that isn’t practical. I’ll start going through the PRs from @Vishwanatha-HD and add technical feedback there, especially around keeping the behavior consistent for canonical little-endian I/O on s390x. If you or the other maintainers eventually want me to break out pieces of my work into smaller, focused PRs, I’m happy to do that — just let me know the direction you prefer and I’ll follow it. Appreciate the guidance! -- 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]
