NobiGo commented on pull request #2600: URL: https://github.com/apache/calcite/pull/2600#issuecomment-962744255
> That is interesting. Tests with "avatica master branch" would fail since now it throws newer exceptions. On the other hand, if we merge this PR, then tests with the released avatica would fail. > > It looks like we should postpone this PR till Avatica is released, and in the mean time we should use a fixed git commit (the one before the updated exceptions) for testing "avatica + calcite" integration. Yes. I have noticed this too. It is unreasonable that test Calcite's master branch with Avatica's master branch directly. I support use a fixed git commit (the one before the updated exceptions) for testing "avatica + calcite" integration. -- 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]
