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
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
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
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
sonarcloud[bot] commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1706875705
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
LakeShen commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1706876930
Hi @NobiGo ,thank you for your review suggestions. I have modified according
to the suggestions. If you have time, please help me review it again
--
This is an automated message from
tanclary commented on code in PR #3391:
URL: https://github.com/apache/calcite/pull/3391#discussion_r1316058041
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -7385,8 +7385,8 @@ private static void checkIf(SqlOperatorFixture f) {
@Test void
asolimando commented on code in PR #3367:
URL: https://github.com/apache/calcite/pull/3367#discussion_r1316122429
##
core/src/main/java/org/apache/calcite/rel/rules/MinusToDistinctRule.java:
##
@@ -0,0 +1,184 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
tanclary merged PR #3302:
URL: https://github.com/apache/calcite/pull/3302
--
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:
sonarcloud[bot] commented on PR #3391:
URL: https://github.com/apache/calcite/pull/3391#issuecomment-1706882873
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
thomasrebele commented on PR #3403:
URL: https://github.com/apache/calcite/pull/3403#issuecomment-1706639183
LGTM
--
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
sonarcloud[bot] commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1703857407
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 #3406:
URL: https://github.com/apache/calcite/pull/3406#issuecomment-1704236810
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
zoov-w commented on PR #3406:
URL: https://github.com/apache/calcite/pull/3406#issuecomment-1704247919
@jiefei30 Thanks, I pushed PR again according to [contribution
rules](https://calcite.apache.org/develop/#contributing).
--
This is an automated message from the Apache Git Service.
sonarcloud[bot] commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1703854702
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
libenchao commented on code in PR #3311:
URL: https://github.com/apache/calcite/pull/3311#discussion_r1311102272
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -4495,6 +4492,27 @@ private boolean advanceRight(TInner right, TKey
rightKey) {
sonarcloud[bot] commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1703857503
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
LakeShen commented on PR #3396:
URL: https://github.com/apache/calcite/pull/3396#issuecomment-1703849466
@kgyrtkirk ,Thank you for your review suggestions. I have modified according
to the suggestions. If you have time, please help me review it again
--
This is an automated message from
zstan commented on code in PR #225:
URL: https://github.com/apache/calcite-avatica/pull/225#discussion_r1314458931
##
core/src/test/java/org/apache/calcite/avatica/util/TimestampFromNumberAccessorTest.java:
##
@@ -101,6 +101,9 @@ public class TimestampFromNumberAccessorTest {
sonarcloud[bot] commented on PR #3405:
URL: https://github.com/apache/calcite/pull/3405#issuecomment-1704732230
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
asolimando commented on code in PR #3367:
URL: https://github.com/apache/calcite/pull/3367#discussion_r1314536702
##
core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java:
##
@@ -2716,11 +2716,25 @@ private void
olivrlee commented on PR #3391:
URL: https://github.com/apache/calcite/pull/3391#issuecomment-1707479064
Commit ready!
--
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
chucheng92 commented on PR #3405:
URL: https://github.com/apache/calcite/pull/3405#issuecomment-1707549985
@NobiGo hi. xiong. thanks for reviewing.
It seems that the commit history has increased a lot due to a force-push by
Tanner. Do I need to rebase the main and then wait to see if
rubenada commented on code in PR #3311:
URL: https://github.com/apache/calcite/pull/3311#discussion_r1316837374
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -4495,6 +4492,27 @@ private boolean advanceRight(TInner right, TKey
rightKey) {
sonarcloud[bot] commented on PR #3408:
URL: https://github.com/apache/calcite/pull/3408#issuecomment-1707463216
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
tanclary merged PR #3391:
URL: https://github.com/apache/calcite/pull/3391
--
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:
libenchao commented on code in PR #3311:
URL: https://github.com/apache/calcite/pull/3311#discussion_r1316730290
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -4495,6 +4492,27 @@ private boolean advanceRight(TInner right, TKey
rightKey) {
olivrlee commented on code in PR #3391:
URL: https://github.com/apache/calcite/pull/3391#discussion_r1316517662
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -7538,8 +7610,8 @@ private static void checkIf(SqlOperatorFixture f) {
@Test void
sonarcloud[bot] commented on PR #3408:
URL: https://github.com/apache/calcite/pull/3408#issuecomment-1707454303
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
libenchao merged PR #3359:
URL: https://github.com/apache/calcite/pull/3359
--
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:
Anthrino opened a new pull request, #3408:
URL: https://github.com/apache/calcite/pull/3408
(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,
tanclary commented on code in PR #3391:
URL: https://github.com/apache/calcite/pull/3391#discussion_r1316550752
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -7538,8 +7610,8 @@ private static void checkIf(SqlOperatorFixture f) {
@Test void
olivrlee commented on code in PR #3391:
URL: https://github.com/apache/calcite/pull/3391#discussion_r1316551091
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -7538,8 +7610,8 @@ private static void checkIf(SqlOperatorFixture f) {
@Test void
LakeShen commented on PR #3367:
URL: https://github.com/apache/calcite/pull/3367#issuecomment-1707512690
Hi @tanclary ,could you help me to review this pr again,if you have any
suggestions, please tell me,If not, please help me to approve,thank you so
much:)
--
This is an automated
macroguo-ghy commented on PR #3389:
URL: https://github.com/apache/calcite/pull/3389#issuecomment-1707559374
>
Hi, @tanclary would you like to merge this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
sonarcloud[bot] commented on PR #3359:
URL: https://github.com/apache/calcite/pull/3359#issuecomment-1707678079
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
LakeShen commented on code in PR #3367:
URL: https://github.com/apache/calcite/pull/3367#discussion_r1313824184
##
core/src/test/java/org/apache/calcite/test/JdbcTest.java:
##
@@ -3773,6 +3774,35 @@ public void checkOrderBy(final boolean desc,
"empid=110;
LakeShen commented on PR #3367:
URL: https://github.com/apache/calcite/pull/3367#issuecomment-1703830009
@tanclary @asolimando @NobiGo ,Thank you for your review suggestions. I have
modified according to the suggestions. If you have time, please help me review
it again:)
--
This is an
rubenada commented on code in PR #3382:
URL: https://github.com/apache/calcite/pull/3382#discussion_r1314595347
##
core/src/main/java/org/apache/calcite/rel/metadata/RelMdRowCount.java:
##
@@ -169,6 +170,12 @@ public Double getRowCount(Calc rel, RelMetadataQuery mq) {
LeonidChistov opened a new pull request, #3407:
URL: https://github.com/apache/calcite/pull/3407
(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
chucheng92 commented on PR #3405:
URL: https://github.com/apache/calcite/pull/3405#issuecomment-1704876596
@NobiGo hi. If you have time, could you help to review it again? thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
sonarcloud[bot] commented on PR #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1704882138
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 #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1704901646
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 #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1704907956
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
rubenada commented on code in PR #3407:
URL: https://github.com/apache/calcite/pull/3407#discussion_r1314852797
##
core/src/main/java/org/apache/calcite/rel/rules/SemiJoinJoinTransposeRule.java:
##
@@ -114,6 +114,18 @@ public SemiJoinJoinTransposeRule(RelBuilderFactory
rubenada commented on code in PR #3407:
URL: https://github.com/apache/calcite/pull/3407#discussion_r1314852797
##
core/src/main/java/org/apache/calcite/rel/rules/SemiJoinJoinTransposeRule.java:
##
@@ -114,6 +114,18 @@ public SemiJoinJoinTransposeRule(RelBuilderFactory
sonarcloud[bot] commented on PR #3367:
URL: https://github.com/apache/calcite/pull/3367#issuecomment-1705201285
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
libenchao commented on code in PR #3311:
URL: https://github.com/apache/calcite/pull/3311#discussion_r1314935681
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -4495,6 +4492,27 @@ private boolean advanceRight(TInner right, TKey
rightKey) {
sonarcloud[bot] commented on PR #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1705477282
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
chucheng92 commented on PR #3395:
URL: https://github.com/apache/calcite/pull/3395#issuecomment-1705506076
@tanclary hi, tanner. I have solved some comments. If you have time, could
you help to review it again? thanks!
--
This is an automated message from the Apache Git Service.
LakeShen commented on PR #3367:
URL: https://github.com/apache/calcite/pull/3367#issuecomment-1705511101
I do not know why there is a Windows pipeline problem, my local MAC
environment is successful
rubenada commented on PR #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1705513136
Thanks for your reactivity @LeonidChistov !
LGTM
I'll merge in 24-48h if no other remark appears.
--
This is an automated message from the Apache Git Service.
To respond to the
sonarcloud[bot] commented on PR #3405:
URL: https://github.com/apache/calcite/pull/3405#issuecomment-1708293514
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
LeonidChistov commented on PR #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1707868065
@rubenada Done.
--
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
rubenada merged PR #3407:
URL: https://github.com/apache/calcite/pull/3407
--
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:
rubenada merged PR #3403:
URL: https://github.com/apache/calcite/pull/3403
--
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:
sonarcloud[bot] commented on PR #3409:
URL: https://github.com/apache/calcite/pull/3409#issuecomment-1708135546
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 #3395:
URL: https://github.com/apache/calcite/pull/3395#issuecomment-1708446685
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-1707826919
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 #3407:
URL: https://github.com/apache/calcite/pull/3407#issuecomment-1707860662
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-1707873463
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
rubenada opened a new pull request, #3409:
URL: https://github.com/apache/calcite/pull/3409
See details in CALCITE-5980
--
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
sonarcloud[bot] commented on PR #3311:
URL: https://github.com/apache/calcite/pull/3311#issuecomment-1707832963
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
NobiGo commented on code in PR #3396:
URL: https://github.com/apache/calcite/pull/3396#discussion_r1315268349
##
core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java:
##
@@ -1254,6 +1254,54 @@ private void
checkSemiOrAntiJoinProjectTranspose(JoinRelType type) {
kgyrtkirk merged PR #3400:
URL: https://github.com/apache/calcite/pull/3400
--
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:
rubenada commented on code in PR #3311:
URL: https://github.com/apache/calcite/pull/3311#discussion_r1315141021
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -4495,6 +4492,27 @@ private boolean advanceRight(TInner right, TKey
rightKey) {
LakeShen commented on code in PR #3396:
URL: https://github.com/apache/calcite/pull/3396#discussion_r1315336549
##
core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java:
##
@@ -1254,6 +1254,54 @@ private void
checkSemiOrAntiJoinProjectTranspose(JoinRelType type) {
LakeShen commented on code in PR #3396:
URL: https://github.com/apache/calcite/pull/3396#discussion_r1315336125
##
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 more
sonarcloud[bot] commented on PR #3311:
URL: https://github.com/apache/calcite/pull/3311#issuecomment-1706161843
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 #3397:
URL: https://github.com/apache/calcite/pull/3397#issuecomment-1709406924
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
JiajunBernoulli commented on PR #3382:
URL: https://github.com/apache/calcite/pull/3382#issuecomment-1709363747
Rename `inputRowCount` and rebase `main`.
@rubenada Thank you~
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
sonarcloud[bot] commented on PR #3404:
URL: https://github.com/apache/calcite/pull/3404#issuecomment-1709544266
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 #3382:
URL: https://github.com/apache/calcite/pull/3382#issuecomment-1709365534
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 #3389:
URL: https://github.com/apache/calcite/pull/3389#issuecomment-1709382851
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
herunkang2018 commented on PR #3389:
URL: https://github.com/apache/calcite/pull/3389#issuecomment-1709299782
@macroguo-ghy LGTM. Remind that there are some duplicated commits compare to
main branch, would you like to rebase from the main branch to fix this?
--
This is an automated
LakeShen commented on code in PR #3415:
URL: https://github.com/apache/calcite/pull/3415#discussion_r1320736268
##
core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java:
##
@@ -565,6 +565,9 @@ void ensureRootConverters() {
}
}
for (RelSubset
sonarcloud[bot] commented on PR #3397:
URL: https://github.com/apache/calcite/pull/3397#issuecomment-1712771007
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 #3397:
URL: https://github.com/apache/calcite/pull/3397#issuecomment-1712875423
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 #3417:
URL: https://github.com/apache/calcite/pull/3417#issuecomment-1712891104
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 #3406:
URL: https://github.com/apache/calcite/pull/3406#issuecomment-1712766095
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 #3415:
URL: https://github.com/apache/calcite/pull/3415#issuecomment-1712751402
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 #3416:
URL: https://github.com/apache/calcite/pull/3416#issuecomment-1712769176
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
LakeShen commented on PR #3414:
URL: https://github.com/apache/calcite/pull/3414#issuecomment-1712769171
Could you add the unit test for you logic?
--
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
asdfgh19 opened a new pull request, #3414:
URL: https://github.com/apache/calcite/pull/3414
jira: [CALCITE-5991](https://issues.apache.org/jira/browse/CALCITE-5991)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
macroguo-ghy opened a new pull request, #3416:
URL: https://github.com/apache/calcite/pull/3416
Simplify 'p OR NOT p' to 'TRUE' (if p is not nullable), or to 'null AND p IS
NOT NULL' (if p is nullable).
GeodeFilter support converting 'p IS NOT NULL' to 'p <> null'.
--
This is an
sonarcloud[bot] commented on PR #3390:
URL: https://github.com/apache/calcite/pull/3390#issuecomment-1712803161
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 #3416:
URL: https://github.com/apache/calcite/pull/3416#issuecomment-1712839355
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
macroguo-ghy opened a new pull request, #3417:
URL: https://github.com/apache/calcite/pull/3417
…abled in BigQuery library)
--
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.
sonarcloud[bot] commented on PR #3414:
URL: https://github.com/apache/calcite/pull/3414#issuecomment-1712749646
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
macroguo-ghy commented on PR #3389:
URL: https://github.com/apache/calcite/pull/3389#issuecomment-1709734955
I have rebased and squashed it. @tanclary
--
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
sonarcloud[bot] commented on PR #3410:
URL: https://github.com/apache/calcite/pull/3410#issuecomment-1710094213
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 #3410:
URL: https://github.com/apache/calcite/pull/3410#issuecomment-1710094291
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
chucheng92 commented on PR #3405:
URL: https://github.com/apache/calcite/pull/3405#issuecomment-1710071252
hi, @NobiGo. If there are no new reviewers to give a comment. Would you help
to merge this? thanks.
--
This is an automated message from the Apache Git Service.
To respond to the
rubenada commented on PR #3311:
URL: https://github.com/apache/calcite/pull/3311#issuecomment-1710023196
Thanks for the review @libenchao .
I'll merge the PR in 24h if no further comments appear.
--
This is an automated message from the Apache Git Service.
To respond to the message,
LakeShen commented on code in PR #3404:
URL: https://github.com/apache/calcite/pull/3404#discussion_r1318408293
##
core/src/test/java/org/apache/calcite/rex/RexExecutorTest.java:
##
@@ -378,4 +383,140 @@ public void run() {
interface Action {
void check(RexBuilder
LakeShen commented on PR #3406:
URL: https://github.com/apache/calcite/pull/3406#issuecomment-1709895369
Hi @zoov-w ,don't begin with fix to the title of a JIRA, should describe the
problem behavior.
BTW, you could rebase the latest code of main branch,the commits and files
changed
sonarcloud[bot] commented on PR #3416:
URL: https://github.com/apache/calcite/pull/3416#issuecomment-1713655572
Kudos, SonarCloud Quality Gate passed! [![Quality Gate
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
rubenada commented on code in PR #3390:
URL: https://github.com/apache/calcite/pull/3390#discussion_r1321146914
##
core/src/test/java/org/apache/calcite/rex/RexExecutorTest.java:
##
@@ -378,4 +378,28 @@ public void run() {
interface Action {
void check(RexBuilder
macroguo-ghy commented on code in PR #3416:
URL: https://github.com/apache/calcite/pull/3416#discussion_r1321367481
##
core/src/main/java/org/apache/calcite/rex/RexSimplify.java:
##
@@ -1993,6 +1993,7 @@ private RexNode simplifyOrs(List terms,
RexUnknownAs unknownAs) {
//
LakeShen commented on PR #3367:
URL: https://github.com/apache/calcite/pull/3367#issuecomment-1713692808
Hi @tanclary ,could you help me to approve this pr if you don't have the
suggestions,thank you so much.
--
This is an automated message from the Apache Git Service.
To respond to the
1 - 100 of 9414 matches
Mail list logo