[GitHub] [calcite] asolimando edited a comment on pull request #2592: [CALCITE-4768] cassandra4

2021-10-27 Thread GitBox
asolimando edited a comment on pull request #2592: URL: https://github.com/apache/calcite/pull/2592#issuecomment-952730479 On a second thought, there are some important info in the PR description that I think should be in the git description as well, could you add it when merging the PR?

[GitHub] [calcite] asolimando edited a comment on pull request #2592: [CALCITE-4768] cassandra4

2021-10-27 Thread GitBox
asolimando edited a comment on pull request #2592: URL: https://github.com/apache/calcite/pull/2592#issuecomment-952638170 Thank you both for the review, I have squashed the commits and updated the commit message, once the test are over (even though there are no code changes since last