etseidl commented on PR #11157: URL: https://github.com/apache/arrow-rs/pull/11157#issuecomment-5819329909
@adriangb I had Codex help me with addressing your review. I believe that C2-C8, C12, C14, N1, and P4 are now addressed. C13 is mostly addressed, but I've left out `union` since it's not needed here and can be added later without breaking. C9, C10, C17, and C18 were mooted by reverting the multi-range change to the push decoder. C1 still needs to be addressed, and I think C11 needs further discussion. PTAL 🙏 -- 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]
