[GitHub] [calcite] sonarcloud[bot] commented on pull request #3399: [CALCITE-5962] Support parse Spark-style syntax LEFT ANTI JOIN in Babel parser

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

[GitHub] [calcite] jiefei30 commented on pull request #3399: [CALCITE-5962] Allow Spark-style syntax LEFT ANTI JOIN in Babel parser

2023-08-30 Thread via GitHub
jiefei30 commented on PR #3399: URL: https://github.com/apache/calcite/pull/3399#issuecomment-1700317394 > @jiefei30 If we only support parse this syntax, Maybe we should use `Support parse Spark-style syntax LEFT ANTI JOIN in Babel parser` . Also need to add parse test in PR.

[GitHub] [calcite] NobiGo commented on pull request #3399: [CALCITE-5962] Allow Spark-style syntax LEFT ANTI JOIN in Babel parser

2023-08-30 Thread via GitHub
NobiGo commented on PR #3399: URL: https://github.com/apache/calcite/pull/3399#issuecomment-1700189732 @jiefei30 If we only support parse this syntax, Maybe we should use `Support parse Spark-style syntax LEFT ANTI JOIN in Babel parser` . Also need to add parse test in PR. -- This is an

[GitHub] [calcite] NobiGo commented on a diff in pull request #3367: [CALCITE-5889] Add the RelRule that converts Minus into UNION ALL..GROUP BY...WHERE

2023-08-30 Thread via GitHub
NobiGo commented on code in PR #3367: URL: https://github.com/apache/calcite/pull/3367#discussion_r1310992460 ## core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java: ## @@ -2699,6 +2699,35 @@ private void

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3387: [CALCITE-5911] Add REGEXP_EXTRACT_ALL function (enabled in BigQuery library)

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

[GitHub] [calcite] Anthrino commented on a diff in pull request #3387: [CALCITE-5911] Add REGEXP_EXTRACT_ALL function (enabled in BigQuery library)

2023-08-30 Thread via GitHub
Anthrino commented on code in PR #3387: URL: https://github.com/apache/calcite/pull/3387#discussion_r1310898531 ## core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java: ## @@ -347,19 +347,119 @@ public static String sha512(ByteString string) { } /** SQL

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3391: [CALCITE-5933] Add SAFE_DIVIDE function (enabled in BigQuery library)

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3391: URL: https://github.com/apache/calcite/pull/3391#issuecomment-1699877299 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 (01467a1797 -> 1eb547c93e)

2023-08-30 Thread tanner
This is an automated email from the ASF dual-hosted git repository. tanner pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git from 01467a1797 [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library) add 1eb547c93e [CALCITE-5910]

[GitHub] [calcite] tanclary merged pull request #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread via GitHub
tanclary merged PR #3252: URL: https://github.com/apache/calcite/pull/3252 -- 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] tanclary merged pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
tanclary merged PR #3369: URL: https://github.com/apache/calcite/pull/3369 -- 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:

[calcite] branch main updated: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread tanner
This is an automated email from the ASF dual-hosted git repository. tanner 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 01467a1797 [CALCITE-5766] Add SAFE_NEGATE function

[GitHub] [calcite] julianhyde commented on pull request #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread via GitHub
julianhyde commented on PR #3252: URL: https://github.com/apache/calcite/pull/3252#issuecomment-1699623273 Two cosmetic things in testSafeNegateFunc: * There are too many '.' in the javadoc. The jira summary does not include a '.' * Function arguments should be indented 4, not

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3391: [CALCITE-5933] Add SAFE_DIVIDE function (enabled in BigQuery library)

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3391: URL: https://github.com/apache/calcite/pull/3391#issuecomment-1699619469 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 #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3252: URL: https://github.com/apache/calcite/pull/3252#issuecomment-1699644767 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 #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3369: URL: https://github.com/apache/calcite/pull/3369#issuecomment-1699627964 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 #3391: [CALCITE-5933] Add SAFE_DIVIDE function (enabled in BigQuery library)

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

[GitHub] [calcite] Anthrino commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
Anthrino commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1310631115 ## core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java: ## @@ -554,6 +555,7 @@ Builder populate() { defineMethod(SPLIT, "split",

[GitHub] [calcite] Anthrino commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
Anthrino commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1310628458 ## testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java: ## @@ -4568,6 +4568,25 @@ private static void checkIf(SqlOperatorFixture f) {

[GitHub] [calcite] Anthrino commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
Anthrino commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1310617255 ## core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java: ## @@ -569,6 +570,8 @@ Builder populate() { defineReflective(PARSE_URL,

[GitHub] [calcite] tanclary commented on pull request #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread via GitHub
tanclary commented on PR #3252: URL: https://github.com/apache/calcite/pull/3252#issuecomment-1699581435 @julianhyde I rebased to include your recent changes and no longer use strings to identify the safe functions' methods. Let me know what you think. -- This is an automated message

[GitHub] [calcite] tanclary commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
tanclary commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1308036102 ## core/src/test/java/org/apache/calcite/test/SqlFunctionsTest.java: ## @@ -260,7 +261,75 @@ static List list() { } catch (RuntimeException e) {

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3252: URL: https://github.com/apache/calcite/pull/3252#issuecomment-1699529860 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 #3252: [CALCITE-5766] Add SAFE_NEGATE function (enabled for BigQuery library)

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

[GitHub] [calcite] tanclary commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
tanclary commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1310568623 ## core/src/test/java/org/apache/calcite/test/SqlFunctionsTest.java: ## @@ -254,18 +254,88 @@ static List list() { final boolean b = f.regexpContains("abc def

[GitHub] [calcite] tanclary commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
tanclary commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1308035240 ## core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java: ## @@ -554,6 +555,7 @@ Builder populate() { defineMethod(SPLIT, "split",

[GitHub] [calcite] tanclary commented on a diff in pull request #3369: [CALCITE-5910] Add REGEXP_EXTRACT and REGEXP_SUBSTR functions (enabled in BigQuery library)

2023-08-30 Thread via GitHub
tanclary commented on code in PR #3369: URL: https://github.com/apache/calcite/pull/3369#discussion_r1310567170 ## core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java: ## @@ -569,6 +570,8 @@ Builder populate() { defineReflective(PARSE_URL,

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3391: [CALCITE-5933] Add SAFE_DIVIDE function (enabled in BigQuery library)

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

[GitHub] [calcite] tanclary commented on a diff in pull request #3389: [CALCITE-5935] Add CODE_POINTS_TO_BYTES function (enabled in BigQuery…

2023-08-30 Thread via GitHub
tanclary commented on code in PR #3389: URL: https://github.com/apache/calcite/pull/3389#discussion_r1310490016 ## core/src/main/java/org/apache/calcite/sql/type/OperandTypes.java: ## @@ -510,6 +510,38 @@ public static SqlOperandTypeChecker variadic( } }; +

[GitHub] [calcite] kgyrtkirk commented on a diff in pull request #3396: [CALCITE-5940] Add the Rules to optimize Limit

2023-08-30 Thread via GitHub
kgyrtkirk commented on code in PR #3396: URL: https://github.com/apache/calcite/pull/3396#discussion_r1310312042 ## core/src/main/java/org/apache/calcite/rel/rules/SortMergeRule.java: ## @@ -0,0 +1,146 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3404: [CALCITE-5968] Provide an interface class for RexExecutable to decouple janino runtime binding

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

[GitHub] [calcite] zhuwenzhuang opened a new pull request, #3404: [CALCITE-5968] Provide an interface class for RexExecutable to decouple janino runtime binding

2023-08-30 Thread via GitHub
zhuwenzhuang opened a new pull request, #3404: URL: https://github.com/apache/calcite/pull/3404 (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

[GitHub] [calcite] herunkang2018 commented on a diff in pull request #3367: [CALCITE-5889] Add the RelRule that converts Minus into UNION ALL..GROUP BY...WHERE

2023-08-30 Thread via GitHub
herunkang2018 commented on code in PR #3367: URL: https://github.com/apache/calcite/pull/3367#discussion_r1308961936 ## core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java: ## @@ -2699,6 +2699,35 @@ private void

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3401: [CALCITE-5965] Avoid unnecessary String concatenations in the RexFieldAccess constructor to improve the performance

2023-08-30 Thread via GitHub
sonarcloud[bot] commented on PR #3401: URL: https://github.com/apache/calcite/pull/3401#issuecomment-1698746657 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 #3403: [CALCITE-5967] UnsupportedOperationException while implementing a call that requires a special collator

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