asolimando edited a comment on pull request #139: URL: https://github.com/apache/calcite-avatica/pull/139#issuecomment-786739740
> Hey @asolimando, I did a few changes on top your work and rebased in https://github.com/zabetak/calcite-avatica/tree/calcite-3163-reb. Can you have a final look? If everything is good I will push it as is. Hi @zabetak , thanks a lot for the changes, it looks much better in this way. I just realized there is a package for "common" test utilities: `package org.apache.calcite.avatica.test`. I feel that `AvaticaMatchers.java`, `IsArrayAccessorResultSetEqual.java` and the other `*TestUtils.java` files would fit better there other than under `org.apache.calcite.avatica.util` where I originally put the test helper classes. In any case LGTM, thanks again! ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
