Re: [PR] [CALCITE-6400] MAP_ENTRIES allows null as a map key [calcite]

2024-05-10 Thread via GitHub
caicancai commented on PR #3787: URL: https://github.com/apache/calcite/pull/3787#issuecomment-2105578522 @mihaibudiu @chucheng92 Good to merge? -- 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

Re: [PR] [CALCITE-6400] MAP_ENTRIES allows null as a map key [calcite]

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

Re: [PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

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

Re: [PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

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

Re: [PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

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

Re: [PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

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

Re: [PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

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

[PR] [CALCITE-6392] Support all PostgreSQL 14 date/time patterns for to_date/to_timestamp [calcite]

2024-05-10 Thread via GitHub
normanj-bitquill opened a new pull request, #3791: URL: https://github.com/apache/calcite/pull/3791 * First phase, mostly reorganizing classes * Does not yet implement to_date or to_timestamp * Fixed up fill mode handling * Isolated the unit test class so that it can make Locale

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597166998 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,34 @@ void assertSubFunReturns(boolean binary, String s, int start,

[PR] [CALCITE-6380] Casts from INTERVAL and STRING to DECIMAL are incorrect [calcite]

2024-05-10 Thread via GitHub
mihaibudiu opened a new pull request, #3790: URL: https://github.com/apache/calcite/pull/3790 (no comment) -- 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,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597164314 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10397,6 +10397,50 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597163878 ## site/_docs/reference.md: ## @@ -2812,6 +2812,7 @@ In the following: | b m p s | MD5(string) | Calculates an MD5 128-bit checksum

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597162522 ## core/src/main/java/org/apache/calcite/sql/type/ReturnTypes.java: ## @@ -551,6 +551,13 @@ public static SqlCall stripSeparator(SqlCall call) { opBinding ->

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597162056 ## core/src/main/java/org/apache/calcite/sql2rel/StandardConvertletTable.java: ## @@ -418,6 +419,32 @@ private static RexNode convertNvl(SqlRexContext cx, SqlCall

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597161501 ## core/src/main/java/org/apache/calcite/sql2rel/StandardConvertletTable.java: ## @@ -418,6 +419,32 @@ private static RexNode convertNvl(SqlRexContext cx, SqlCall

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597157484 ## core/src/main/java/org/apache/calcite/sql/type/ReturnTypes.java: ## @@ -551,6 +551,13 @@ public static SqlCall stripSeparator(SqlCall call) { opBinding ->

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
julianhyde commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1597156525 ## core/src/main/java/org/apache/calcite/sql/SqlKind.java: ## @@ -421,9 +421,12 @@ public enum SqlKind { /** {@code DECODE} function (Oracle). */ DECODE, -

Re: [PR] [CALCITE-6322] Casts to DECIMAL types are ignored [calcite]

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

Re: [PR] [CALCITE-6322] Casts to DECIMAL types are ignored [calcite]

2024-05-10 Thread via GitHub
mihaibudiu commented on PR #3733: URL: https://github.com/apache/calcite/pull/3733#issuecomment-2105146098 @zabetak I think I have addressed your comments. I moved part of this PR into in a separate one which has been merged https://github.com/apache/calcite/pull/3779. So what's left

(calcite) branch main updated: [CALCITE-6389] RexBuilder.removeCastFromLiteral does not preserve semantics for some types of literal

2024-05-10 Thread mbudiu
This is an automated email from the ASF dual-hosted git repository. mbudiu 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 c228804e25 [CALCITE-6389]

Re: [PR] [CALCITE-6389] RexBuilder.removeCastFromLiteral does not preserve semantics for some… [calcite]

2024-05-10 Thread via GitHub
mihaibudiu merged PR #3779: URL: https://github.com/apache/calcite/pull/3779 -- 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-6389] RexBuilder.removeCastFromLiteral does not preserve semantics for some… [calcite]

2024-05-10 Thread via GitHub
snuyanzin commented on code in PR #3779: URL: https://github.com/apache/calcite/pull/3779#discussion_r1597039588 ## core/src/main/java/org/apache/calcite/sql/type/SqlTypeUtil.java: ## @@ -1822,6 +1823,35 @@ public static RelDataType extractLastNFields(RelDataTypeFactory

Re: [PR] [CALCITE-6389] RexBuilder.removeCastFromLiteral does not preserve semantics for some… [calcite]

2024-05-10 Thread via GitHub
mihaibudiu commented on code in PR #3779: URL: https://github.com/apache/calcite/pull/3779#discussion_r1597033016 ## core/src/main/java/org/apache/calcite/sql/type/SqlTypeUtil.java: ## @@ -1822,6 +1823,35 @@ public static RelDataType extractLastNFields(RelDataTypeFactory

(calcite) branch main updated: [CALCITE-6365] Support for RETURNING clause of JSON_QUERY

2024-05-10 Thread mbudiu
This is an automated email from the ASF dual-hosted git repository. mbudiu 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 ea441e7c75 [CALCITE-6365] Support for RETURNING

Re: [PR] [CALCITE-6365] Support for RETURNING clause of JSON_QUERY [calcite]

2024-05-10 Thread via GitHub
mihaibudiu merged PR #3771: URL: https://github.com/apache/calcite/pull/3771 -- 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-6358] Support all PostgreSQL 14 date/time patterns [calcite]

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

Re: [PR] [CALCITE-6358] Support all PostgreSQL 14 date/time patterns [calcite]

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

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

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

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on PR #3785: URL: https://github.com/apache/calcite/pull/3785#issuecomment-2104872020 @mihaibudiu I would like to ask a question. The type check of NVL2 (2.0, 'abc', true) in calcite is allowed. The string type and the boolean type are comparable. I am not sure about

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596944009 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596942382 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596920047 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596920047 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596920047 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6397] Add NVL2 function (enabled in Spark library) [calcite]

2024-05-10 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1596920047 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -10036,6 +10036,45 @@ void assertSubFunReturns(boolean binary, String s, int start,

Re: [PR] [CALCITE-6365] Support for RETURNING clause of JSON_QUERY [calcite]

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

Re: [PR] [CALCITE-6365] Support for RETURNING clause of JSON_QUERY [calcite]

2024-05-10 Thread via GitHub
dawidwys commented on PR #3771: URL: https://github.com/apache/calcite/pull/3771#issuecomment-2104173057 Thank you both for the reviews @snuyanzin @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