rubenada commented on PR #313: URL: https://github.com/apache/calcite-avatica/pull/313#issuecomment-5127905620
Thanks a lot for the review @potiuk ! Regarding your comments: 1. This model does **not** cover `apache/calcite-avatica-go`, Avatica-go would require its own model, and for the moment this is out of the scope. 2. I've added your proposed clause, please take a look. 3. Well spotted. Fixed. 4. I double checked, and actually that CVE was related to Calcite, not Avatica, so I've removed it. 5. I've removed that "status" mention, so that it's ready when we decide to merge this; but I'd like to have at least one other PMC approval. -- 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]
