vlsi commented on pull request #2600:
URL: https://github.com/apache/calcite/pull/2600#issuecomment-963081240


   I'm inclined we should solve this at `.iq` level or rewrite `.iq` test to 
regular JUnit, so can support multiple Avatica versions (e.g. via "contains 
message1 or message2").
   
   Otherwise, it would be the case we support one specific Avatica version 
only, which questions the split of Calcite and Avatica into different 
repositories. So we either support at least two Avatica versions, or we should 
just merge the repositories and stop pretending they are independent libraries.
   
   


-- 
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]


Reply via email to