[calcite-avatica-go] branch main updated: Update docker release script to use rc tag in vote email links

2022-10-13 Thread francischuang
This is an automated email from the ASF dual-hosted git repository. francischuang pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git The following commit(s) were added to refs/heads/main by this push: new d39ca14 Update docker release sc

[GitHub] [calcite] rubenada commented on pull request #2937: [CALCITE-5332] Configuring PruneEmptyRules is cumbersome

2022-10-13 Thread GitBox
rubenada commented on PR #2937: URL: https://github.com/apache/calcite/pull/2937#issuecomment-1277715384 @zabetak IMO the change makes sense, lgtm. Have you checked if there are other rules that could benefit from the same refactoring? -- This is an automated message from the Apache Gi

[GitHub] [calcite] jaystarshot commented on a diff in pull request #2931: [CALCITE-5248] Support extended column aliasing in select list and where, on and filter clauses

2022-10-13 Thread GitBox
jaystarshot commented on code in PR #2931: URL: https://github.com/apache/calcite/pull/2931#discussion_r994622813 ## core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java: ## @@ -2815,6 +2815,45 @@ private SqlDialect nonOrdinalDialect() { sql(query).o

[GitHub] [calcite] jaystarshot commented on a diff in pull request #2931: [CALCITE-5248] Support extended column aliasing in select list and where, on and filter clauses

2022-10-13 Thread GitBox
jaystarshot commented on code in PR #2931: URL: https://github.com/apache/calcite/pull/2931#discussion_r994622813 ## core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java: ## @@ -2815,6 +2815,45 @@ private SqlDialect nonOrdinalDialect() { sql(query).o