[calcite] 01/03: [CALCITE-5730] Initial null values can be dropped by EnumerableLimitSort with offset

2023-06-02 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git commit e34caf70ac97996035b8fd8fc494a64022b07f91 Author: guofeng.my AuthorDate: Thu Jun 1 17:06:50 2023 +0800 [CALCITE-5730]

[calcite] 02/03: [CALCITE-5714] Add MAP_ENTRIES function (enabled in Spark library)

2023-06-02 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git commit 671f49d7d0ba09559a7d4977ebf250259a6fe526 Author: yongen.ly AuthorDate: Fri May 19 17:37:09 2023 +0800 [CALCITE-5714]

[calcite] 03/03: [CALCITE-5722] Sarg.isComplementedPoints fails with anti-points which are equal under `compareTo` but not `equals`

2023-06-02 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git commit 6ffcb860528e9025398b7b756fd4819fbe781e31 Author: ian.bertolacci AuthorDate: Wed May 24 12:49:40 2023 -0700

[calcite] branch main updated (8ea4160f10 -> 6ffcb86052)

2023-06-02 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git discard 8ea4160f10 [CALCITE-5722] Sarg.isComplementedPoints fails with anti-points which are equal under `compareTo` but not

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3054: (do not check in)

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

[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3202: [CALCITE-5704] Add ARRAY_EXCEPT, ARRAY_INTERSECT, ARRAY_UNION function (enabled in Spark library)

2023-06-02 Thread via GitHub
JiajunBernoulli commented on code in PR #3202: URL: https://github.com/apache/calcite/pull/3202#discussion_r1215049429 ## site/_docs/reference.md: ## @@ -2653,11 +2653,14 @@ BigQuery's type system uses confusingly different names for types and functions: | * | ACOSH(numeric)

[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3202: [CALCITE-5704] Add ARRAY_EXCEPT, ARRAY_INTERSECT, ARRAY_UNION function (enabled in Spark library)

2023-06-02 Thread via GitHub
JiajunBernoulli commented on code in PR #3202: URL: https://github.com/apache/calcite/pull/3202#discussion_r1215055311 ## site/_docs/reference.md: ## @@ -2653,11 +2653,14 @@ BigQuery's type system uses confusingly different names for types and functions: | * | ACOSH(numeric)

[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3202: [CALCITE-5704] Add ARRAY_EXCEPT, ARRAY_INTERSECT, ARRAY_UNION function (enabled in Spark library)

2023-06-02 Thread via GitHub
JiajunBernoulli commented on code in PR #3202: URL: https://github.com/apache/calcite/pull/3202#discussion_r1215049429 ## site/_docs/reference.md: ## @@ -2653,11 +2653,14 @@ BigQuery's type system uses confusingly different names for types and functions: | * | ACOSH(numeric)

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3054: (do not check in)

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

[GitHub] [calcite] julianhyde commented on a diff in pull request #3206: [CALCITE-5724] Generated SQL uses literal values in ORDER BY clauses

2023-06-02 Thread via GitHub
julianhyde commented on code in PR #3206: URL: https://github.com/apache/calcite/pull/3206#discussion_r1214942298 ## core/src/main/java/org/apache/calcite/rel/rel2sql/SqlImplementor.java: ## @@ -1119,6 +1119,34 @@ public List fieldList() { }; } +/** + *

[calcite] branch main updated: [CALCITE-5722] Sarg.isComplementedPoints fails with anti-points which are equal under `compareTo` but not `equals`

2023-06-02 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde 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 8ea4160f10 [CALCITE-5722] Sarg.isComplementedPoints

[GitHub] [calcite] asfgit closed pull request #3224: [CALCITE-5722] RangeSets.isPoint incorrectly gives false for closed ranges with endpoints which are equal under `.compareTo` but not under `.equals

2023-06-02 Thread via GitHub
asfgit closed pull request #3224: [CALCITE-5722] RangeSets.isPoint incorrectly gives false for closed ranges with endpoints which are equal under `.compareTo` but not under `.equals` URL: https://github.com/apache/calcite/pull/3224 -- This is an automated message from the Apache Git

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3206: [CALCITE-5724] Generated SQL uses literal values in ORDER BY clauses

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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3206: [CALCITE-5724] Generated SQL uses literal values in ORDER BY clauses

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

[GitHub] [calcite] liuyongvs commented on pull request #3236: [CALCITE-5738] Add SORT_ARRAY function (enabled in Spark library)

2023-06-02 Thread via GitHub
liuyongvs commented on PR #3236: URL: https://github.com/apache/calcite/pull/3236#issuecomment-1573906121 fix conflicts with main branch and squash the commits -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3236: [CALCITE-5738] Add SORT_ARRAY function (enabled in Spark library)

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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3238: [CALCITE-5744] Add STR_TO_MAP function (enabled in Spark library)

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

[GitHub] [calcite] snuyanzin commented on pull request #3113: [CALCITE-5567] Enable jdk19 in ci

2023-06-02 Thread via GitHub
snuyanzin commented on PR #3113: URL: https://github.com/apache/calcite/pull/3113#issuecomment-1573841516 thanks a lot for having a look @zabetak , @rubenada if there is no more objections/suggestions i'm going to merge this weekend -- This is an automated message from the Apache

[GitHub] [calcite] snuyanzin commented on a diff in pull request #3113: [CALCITE-5567] Enable jdk19 in ci

2023-06-02 Thread via GitHub
snuyanzin commented on code in PR #3113: URL: https://github.com/apache/calcite/pull/3113#discussion_r1214460250 ## core/src/main/java/org/apache/calcite/util/Util.java: ## @@ -1717,16 +1719,14 @@ private static void appendPosixTime(StringBuilder buf, int millis) { *

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3212: [CALCITE-5710] Add ARRAY_MAX, ARRAY_MIN function (enabled in Spark li…

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

[GitHub] [calcite] liuyongvs commented on pull request #3212: [CALCITE-5710] Add ARRAY_MAX, ARRAY_MIN function (enabled in Spark li…

2023-06-02 Thread via GitHub
liuyongvs commented on PR #3212: URL: https://github.com/apache/calcite/pull/3212#issuecomment-1573782725 @JiajunBernoulli git rebase to fix conflicts with main branch -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [calcite] liuyongvs commented on a diff in pull request #3212: [CALCITE-5710] Add ARRAY_MAX, ARRAY_MIN function (enabled in Spark li…

2023-06-02 Thread via GitHub
liuyongvs commented on code in PR #3212: URL: https://github.com/apache/calcite/pull/3212#discussion_r1214409643 ## core/src/main/java/org/apache/calcite/sql/type/ReturnTypes.java: ## @@ -500,7 +515,7 @@ public static SqlCall stripSeparator(SqlCall call) { * Returns the

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3113: [CALCITE-5567] Enable jdk19 in ci

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

[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #3212: [CALCITE-5710] Add ARRAY_MAX, ARRAY_MIN function (enabled in Spark li…

2023-06-02 Thread via GitHub
JiajunBernoulli commented on code in PR #3212: URL: https://github.com/apache/calcite/pull/3212#discussion_r1214362380 ## core/src/main/java/org/apache/calcite/sql/type/ReturnTypes.java: ## @@ -500,7 +515,7 @@ public static SqlCall stripSeparator(SqlCall call) { * Returns

[calcite] branch main updated: [CALCITE-5714] Add MAP_ENTRIES function (enabled in Spark library)

2023-06-02 Thread jiajunxie
This is an automated email from the ASF dual-hosted git repository. jiajunxie 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 c14e071590 [CALCITE-5714] Add MAP_ENTRIES

[GitHub] [calcite] JiajunBernoulli merged pull request #3218: [CALCITE-5714] Add MAP_ENTRIES function (enabled in Spark library)

2023-06-02 Thread via GitHub
JiajunBernoulli merged PR #3218: URL: https://github.com/apache/calcite/pull/3218 -- 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:

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3113: [CALCITE-5567] Enable jdk19 in ci

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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3238: [CALCITE-5744] Add STR_TO_MAP function (enabled in Spark library)

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

[GitHub] [calcite] liuyongvs commented on pull request #3238: [CALCITE-5744] Add STR_TO_MAP function (enabled in Spark library)

2023-06-02 Thread via GitHub
liuyongvs commented on PR #3238: URL: https://github.com/apache/calcite/pull/3238#issuecomment-1573584337 hi @JiajunBernoulli @tanclary @MasseGuillaume do you have time to review this ,just review STR_TO_MAP and after the former commits merged, i will rebase it.otherwise, it will lots

[GitHub] [calcite] liuyongvs commented on pull request #3239: [CALCITE-5739] Add MAP_FROM_ARRAYS function (enabled in Spark library)

2023-06-02 Thread via GitHub
liuyongvs commented on PR #3239: URL: https://github.com/apache/calcite/pull/3239#issuecomment-1573582884 hi @JiajunBernoulli @tanclary @MasseGuillaume do you have time to review this ,just review MAP_FROM_ARRAYS and after the MAP_ENTRIES merged, i will rebase it.otherwise, it will lots

[GitHub] [calcite] liuyongvs opened a new pull request, #3239: [CALCITE-5739] Add MAP_FROM_ARRAYS function (enabled in Spark library)

2023-06-02 Thread via GitHub
liuyongvs opened a new pull request, #3239: URL: https://github.com/apache/calcite/pull/3239 MAP_FROM_ARRAYS (array1, array2) Returns a map created from an array1 and *array2. Note that the lengths of two arrays should be the same ``` > SELECT map_from_arrays(array(1.0,

[GitHub] [calcite] liuyongvs opened a new pull request, #3238: [CALCITE-5744] Add STR_TO_MAP function (enabled in Spark library)

2023-06-02 Thread via GitHub
liuyongvs opened a new pull request, #3238: URL: https://github.com/apache/calcite/pull/3238  STR_TO_MAP(string[, stringDelimiter[, keyValueDelimiter]]) Returns a map after splitting the string into key/value pairs using delimiters. Default delimiters are ',' for stringDelimiter and

[GitHub] [calcite] zstan commented on a diff in pull request #3211: [CALCITE-5708] Change SUBSTRING result if either of parameters is NULL

2023-06-02 Thread via GitHub
zstan commented on code in PR #3211: URL: https://github.com/apache/calcite/pull/3211#discussion_r1214203114 ## core/src/test/java/org/apache/calcite/test/SqlValidatorTest.java: ## @@ -1042,6 +1042,10 @@ void testDyadicCollateOperator() { .columnType("VARCHAR(1) NOT

[GitHub] [calcite] zabetak commented on a diff in pull request #3113: [CALCITE-5567] Enable jdk19 in ci

2023-06-02 Thread via GitHub
zabetak commented on code in PR #3113: URL: https://github.com/apache/calcite/pull/3113#discussion_r1214176911 ## core/src/main/java/org/apache/calcite/util/Util.java: ## @@ -1717,16 +1719,14 @@ private static void appendPosixTime(StringBuilder buf, int millis) { * @return

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3237: [CALCITE-5728] Add ARRAY_TO_STRING function (enabled in BigQuery library)

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

[GitHub] [calcite] zoudan opened a new pull request, #3237: [CALCITE-5728] Add ARRAY_TO_STRING function (enabled in BigQuery library)

2023-06-02 Thread via GitHub
zoudan opened a new pull request, #3237: URL: https://github.com/apache/calcite/pull/3237 (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,

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3113: [CALCITE-5567] Enable jdk19 in ci

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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3207: [CALCITE-5707] Add ARRAY_CONTAINS function (enabled in Spark library)

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

[calcite] branch main updated: [CALCITE-5730] Initial null values can be dropped by EnumerableLimitSort with offset

2023-06-02 Thread rubenql
This is an automated email from the ASF dual-hosted git repository. rubenql 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 295df907d1 [CALCITE-5730] Initial null values

[GitHub] [calcite] rubenada merged pull request #3228: [CALCITE-5730] Initial null values can be dropped by EnumerableLimitSort with offset

2023-06-02 Thread via GitHub
rubenada merged PR #3228: URL: https://github.com/apache/calcite/pull/3228 -- 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:

[GitHub] [calcite] snuyanzin commented on a diff in pull request #3113: [CALCITE-5567] Enable jdk19 in ci

2023-06-02 Thread via GitHub
snuyanzin commented on code in PR #3113: URL: https://github.com/apache/calcite/pull/3113#discussion_r1213976159 ## site/_docs/history.md: ## @@ -43,6 +43,10 @@ z. Breaking Changes {: #breaking-1-35-0} +The way of Locale parsing changed within

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3212: [CALCITE-5710] Add ARRAY_MAX, ARRAY_MIN function (enabled in Spark li…

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