ywc88 commented on PR #1099: URL: https://github.com/apache/arrow-adbc/pull/1099#issuecomment-1743863928
Sorry @OleMussmann for the delayed review, I was out sick most of last week and didn't get a chance to look at this until today. Other than a minor nit which I left a comment on, I appreciate how comprehensive your tests are and don't see anything that could be an issue (e.g. resource leak) after checking this PR out locally and playing with it. @lidavidm could you take a second look just in case I missed something? -- 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]
