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

2024-05-31 Thread via GitHub
mihaibudiu merged PR #3785: URL: https://github.com/apache/calcite/pull/3785 -- 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: commits-unsubscr...@calcite

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

2024-05-31 Thread via GitHub
caicancai commented on PR #3785: URL: https://github.com/apache/calcite/pull/3785#issuecomment-2141399389 @mihaibudiu @julianhyde @YiwenWu Thanks for the review, squashed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

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

2024-05-29 Thread via GitHub
caicancai commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1619142488 ## 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-6397] Add NVL2 function (enabled in Oracle, Spark library) [calcite]

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

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

2024-05-29 Thread via GitHub
YiwenWu commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1618723448 ## 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-6397] Add NVL2 function (enabled in Oracle, Spark library) [calcite]

2024-05-28 Thread via GitHub
caicancai commented on PR #3785: URL: https://github.com/apache/calcite/pull/3785#issuecomment-2136377502 @YiwenWu If you have time, can you help me take a look at this PR? Thank you -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

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

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

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

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

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

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

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

2024-05-14 Thread via GitHub
mihaibudiu commented on code in PR #3785: URL: https://github.com/apache/calcite/pull/3785#discussion_r1600522936 ## 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 Oracle, Spark library) [calcite]

2024-05-14 Thread via GitHub
caicancai commented on PR #3785: URL: https://github.com/apache/calcite/pull/3785#issuecomment-2110531888 @mihaibudiu PATL -- 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. T

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

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

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

2024-05-13 Thread via GitHub
caicancai commented on PR #3785: URL: https://github.com/apache/calcite/pull/3785#issuecomment-2107959410 @mihaibudiu Hello, I opened jira to discuss the comparability of character and boolean types because I'm not sure if this is a bug. https://issues.apache.org/jira/browse/CALCITE-6409