vlsi commented on pull request #2600: URL: https://github.com/apache/calcite/pull/2600#issuecomment-963069411
This is the first time we have had an intended failure like that, and what do you think of a temporary disabling `Avatica master+current Calcite` job? As Avatica is released, we could re-enable the check. Another option is to ask @julianhyde or someone who knows Quidem the way to handle the case in `.iq` test somehow (e.g. use different assertions for different avatica versions). I do not know what is the way to make `.iq` conditional on Avatica version. -- 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]
