matthewgapp commented on PR #8839: URL: https://github.com/apache/arrow-datafusion/pull/8839#issuecomment-1899086866
> Thank you @matthewgapp -- I think this PR is basically ready to go. I took the liberty of merging up from main to resolve a conflict and some more comments in [675a6df](https://github.com/apache/arrow-datafusion/commit/675a6df8eb2de2a8bacc7359b2c7b73c24b3831f) > > I also have a few suggestions about error messages and naming but I don't think any of them are required on this PR (we could potentially do them (or not) as a follow on PR) Thanks for the review and for layering in those tweaks. I'll layer in changes based on your comments real quick and then should be good 2 go. 🚢 -- 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]
