Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-03-12 Thread via GitHub
snuyanzin merged PR #3569: URL: https://github.com/apache/calcite/pull/3569 -- 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:

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-03-06 Thread via GitHub
mihaibudiu commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1981243084 No objections from me -- 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

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-03-06 Thread via GitHub
snuyanzin commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1980905984 @mihaibudiu since we are going to have a release soon I would merge this by the end of this week please let me know if you have any objections -- This is an automated message from

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-02-23 Thread via GitHub
sonarcloud[bot] commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1962229883 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-02-23 Thread via GitHub
mihaibudiu commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-196177 I am interpreting the comments as an approval, so I plan to merge this soon. Let me know if you have objections. -- This is an automated message from the Apache Git Service. To

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-01-26 Thread via GitHub
julianhyde commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1912676830 +1 Thanks for your work, @mihaibudiu! -- 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

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-01-26 Thread via GitHub
julianhyde commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1468129736 ## core/src/main/java/org/apache/calcite/sql/type/SqlTypeName.java: ## @@ -309,7 +314,10 @@ public enum SqlTypeName { * matches the given name, or throws {@link

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2024-01-25 Thread via GitHub
mihaibudiu commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1911210533 This PR is ready for a review. It adds a feature to Calcite which has been requested 9 years ago. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
sonarcloud[bot] commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1869755221 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
mihaibudiu commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1869743676 Thank you @tanclary for the review, I have tried to address your suggestions in the last commit. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
tanclary commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1436584354 ## cassandra/src/main/java/org/apache/calcite/adapter/cassandra/CassandraFilter.java: ## @@ -192,6 +193,9 @@ private String translateMatch(RexNode condition) {

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
mihaibudiu commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1436583927 ## core/src/main/java/org/apache/calcite/sql/type/SqlTypeName.java: ## @@ -309,7 +314,10 @@ public enum SqlTypeName { * matches the given name, or throws {@link

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
mihaibudiu commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1436582231 ## core/src/main/java/org/apache/calcite/sql/type/SqlTypeName.java: ## @@ -65,12 +65,16 @@ public enum SqlTypeName { DATE(PrecScale.NO_NO, false, Types.DATE,

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-26 Thread via GitHub
mihaibudiu commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1436581647 ## cassandra/src/main/java/org/apache/calcite/adapter/cassandra/CassandraFilter.java: ## @@ -192,6 +193,9 @@ private String translateMatch(RexNode condition) {

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-24 Thread via GitHub
tanclary commented on code in PR #3569: URL: https://github.com/apache/calcite/pull/3569#discussion_r1435825219 ## cassandra/src/main/java/org/apache/calcite/adapter/cassandra/CassandraFilter.java: ## @@ -192,6 +193,9 @@ private String translateMatch(RexNode condition) {

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

2023-12-21 Thread via GitHub
mihaibudiu commented on PR #3569: URL: https://github.com/apache/calcite/pull/3569#issuecomment-1867088799 I have simplified the naming of the types, as suggested by @julianhyde in a JIRA discussion for the uber-issue 208 that this PR is a part of:

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

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

Re: [PR] [CALCITE-6138] Parser does not accept TIMESTAMP WITH TIME ZONE as a data type [calcite]

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