kumarlokesh commented on PR #7434: URL: https://github.com/apache/arrow-rs/pull/7434#issuecomment-2893033443
> Thank you @kumarlokesh -- I think this looks really nicely documented and tested. > > I had a few API questions / suggestions but I don't think they are necessary. > > It seems to me that this PR does not change any APIs (only adds new ones) so I think we could merge it before the next major release. What do you think? > > Thanks again for pushing this along and breaking it up into smaller chunks @alamb thanks for the review! Yes, I think this PR could be merged before the next major release. Addressed review comments here https://github.com/apache/arrow-rs/pull/7434/commits/cf50b5b851ce2e1cb557e5ab3cdb16f6806663e3 -- 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]
