alamb commented on PR #7535: URL: https://github.com/apache/arrow-rs/pull/7535#issuecomment-2925695138
> LGTM, except a few older review comments might have been missed? Also a bunch of nits to consider either before merge or as follow-up work. Yeah, I think it is getting close to impossible to follow along with all the comments on this PR: <img width="1261" alt="Screenshot 2025-05-31 at 3 16 47 PM" src="https://github.com/user-attachments/assets/6a73edf4-ea5d-4800-8079-d60189d7fdb9"; /> Thus, let's bias towards action (and knowing we can always change code), I am going to merge it so we can keep hacking / improving in follow on PRs. Thank you so much @mkarbo and @scovich for pushing this along Here is to some more exciting PRs ahead: 🚀 -- 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
