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
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.
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
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
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
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
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
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]
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:
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:
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
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
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
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
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
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
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",
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) {
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,
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
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) {
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
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
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
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",
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,
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
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(
}
};
+
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
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
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
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
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
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
34 matches
Mail list logo