avantgardnerio commented on PR #4055: URL: https://github.com/apache/arrow-rs/pull/4055#issuecomment-1505437476
> NVM, I had a buggy ODBC driver This is why I think integration testing is almost as important as manually trying to interpret the spec. IMO we should try stuff with the JDBC / ODBC drivers, record what happens, then try to mimic those tests with the Rust client and add them to the test suite. I wouldn't want to merge the change that breaks something about JDBC that others are actively using. In the case of `CommandStatementQuery` specifically, I wouldn't be surprised if that isn't working properly, since I've only been testing with JDBC and that driver always seems to turn everything into a prepared statement, even if on the Java side the user asks for an un-prepared statement. -- 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]
