[GitHub] [calcite] sonarcloud[bot] commented on pull request #3039: [CALCITE-5180] DATE/TIME type alias + constructors

2023-01-20 Thread sonarcloud
sonarcloud[bot] commented on PR #3039: URL: https://github.com/apache/calcite/pull/3039#issuecomment-1399008135 Kudos, SonarCloud Quality Gate passed! [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png

[GitHub] [calcite] birschick-bq commented on a diff in pull request #3038: [CALCITE-5483] ProjectAggregateMergeRule throws exception if literal is non-numeric

2023-01-20 Thread via GitHub
birschick-bq commented on code in PR #3038: URL: https://github.com/apache/calcite/pull/3038#discussion_r1082986380 ## core/src/main/java/org/apache/calcite/rel/rules/ProjectAggregateMergeRule.java: ## @@ -99,25 +100,27 @@ && kindCount(project.getProjects(), SqlKind.CASE) == 0)

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3037: [CALCITE-5446] TIMESTAMP_WITH_LOCAL_TIMEZONE in JDBC

2023-01-20 Thread sonarcloud
sonarcloud[bot] commented on PR #3037: URL: https://github.com/apache/calcite/pull/3037#issuecomment-1398850502 Kudos, SonarCloud Quality Gate passed! [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png

[GitHub] [calcite-avatica] julianhyde commented on pull request #205: [CALCITE-5487] Proper semantics for TIMESTAMP WITH LOCAL TIME ZONE

2023-01-20 Thread via GitHub
julianhyde commented on PR #205: URL: https://github.com/apache/calcite-avatica/pull/205#issuecomment-1398777208 If it's not too much trouble, could you refactor the tests as a commit *before* the fix? It's disconcerting to see lots of test changes in a bug fix; I'd prefer to just see

[GitHub] [calcite] birschick-bq closed pull request #3031: [CALCITE-5483] ProjectAggregateMergeRule throws exception if literal is non-numeric

2023-01-20 Thread GitBox
birschick-bq closed pull request #3031: [CALCITE-5483] ProjectAggregateMergeRule throws exception if literal is non-numeric URL: https://github.com/apache/calcite/pull/3031 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[calcite-site] branch main updated: Website deployed from calcite@d2da6532c9956462501af1d8337c680e4fa4782d

2023-01-20 Thread asf-ci-deploy
This is an automated email from the ASF dual-hosted git repository. asf-ci-deploy pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite-site.git The following commit(s) were added to refs/heads/main by this push: new ee1756b1 Website deployed from

[calcite] branch site updated: Site: Switch PMC Chair to Stamatis Zampetakis

2023-01-20 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch site in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/site by this push: new 9089bacae2 Site: Switch PMC Chair to Stamatis

[calcite] branch main updated: Site: Switch PMC Chair to Stamatis Zampetakis

2023-01-20 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/main by this push: new d2da6532c9 Site: Switch PMC Chair to Stamatis