[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

[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] 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] 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] sonarcloud[bot] commented on pull request #3396: [CALCITE-5940] Add the Rules to optimize Limit

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] tanclary merged pull request #3302: [CALCITE-5821] Add FORMAT_NUMBER function (enabled in Hive and Spark library)

2023-09-05 Thread via GitHub
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:

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

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] thomasrebele commented on pull request #3403: [CALCITE-5967] UnsupportedOperationException while implementing a call that requires a special collator

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3396: [CALCITE-5940] Add the Rules to optimize Limit

2023-09-02 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3406: [CALCITE-5974] fix bug when es index mapping set dynamic_templates without properties

2023-09-03 Thread via GitHub
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

[GitHub] [calcite] zoov-w commented on pull request #3406: [CALCITE-5974] fix bug when es index mapping set dynamic_templates without properties

2023-09-03 Thread via GitHub
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.

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3396: [CALCITE-5940] Add the Rules to optimize Limit

2023-09-02 Thread via GitHub
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

[GitHub] [calcite] libenchao commented on a diff in pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-02 Thread via GitHub
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) {

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3396: [CALCITE-5940] Add the Rules to optimize Limit

2023-09-02 Thread via GitHub
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

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

2023-09-02 Thread via GitHub
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

[GitHub] [calcite-avatica] zstan commented on a diff in pull request #225: [CALCITE-5957] Date representation string without precede 0 is acceptable

2023-09-03 Thread via GitHub
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 {

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3405: [CALCITE-5961] Type inference of ARRAY_COMPACT is incorrect

2023-09-04 Thread via GitHub
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

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

2023-09-04 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] chucheng92 commented on pull request #3405: [CALCITE-5961] Type inference of ARRAY_COMPACT is incorrect

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on a diff in pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-06 Thread via GitHub
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) {

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3408: [CALCITE-5978] Add REGEXP_INSTR function (enabled in BigQuery library)

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] tanclary merged pull request #3391: [CALCITE-5933] Add SAFE_DIVIDE function (enabled in BigQuery library)

2023-09-05 Thread via GitHub
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:

[GitHub] [calcite] libenchao commented on a diff in pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-05 Thread via GitHub
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) {

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

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3408: [CALCITE-5978] Add REGEXP_INSTR function (enabled in BigQuery library)

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] libenchao merged pull request #3359: [CALCITE-5905] Documentation for CREATE TYPE is incorrect

2023-09-06 Thread via GitHub
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:

[GitHub] [calcite] Anthrino opened a new pull request, #3408: [CALCITE-5978] Add REGEXP_INSTR function (enabled in BigQuery library)

2023-09-05 Thread via GitHub
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,

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

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

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

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3359: [CALCITE-5905] Documentation for CREATE TYPE is incorrect

2023-09-05 Thread via GitHub
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

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

2023-09-02 Thread via GitHub
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;

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

2023-09-02 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on a diff in pull request #3382: [CALCITE-5944] Add metadata for Sample

2023-09-04 Thread via GitHub
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) {

[GitHub] [calcite] LeonidChistov opened a new pull request, #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] chucheng92 commented on pull request #3405: [CALCITE-5961] Type inference of ARRAY_COMPACT is incorrect

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on a diff in pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on a diff in pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

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

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] libenchao commented on a diff in pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-04 Thread via GitHub
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) {

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3407: [CALCITE-5952] Semi-Join incorrectly reordered with Left-Join by SemiJoinJoinTransposeRule

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] chucheng92 commented on pull request #3395: [CALCITE-5948] Explicit casting should be made if the type of an element in ARRAY/MAP not equals with the derived component type

2023-09-04 Thread via GitHub
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.

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

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on pull request #3407: [CALCITE-5952] SemiJoinJoinTransposeRule should check if JoinType supports pushing predicates into its inputs

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3405: [CALCITE-5961] Type inference of ARRAY_COMPACT is incorrect

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] LeonidChistov commented on pull request #3407: [CALCITE-5952] SemiJoinJoinTransposeRule should check if JoinType supports pushing predicates into its inputs

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] rubenada merged pull request #3407: [CALCITE-5952] SemiJoinJoinTransposeRule should check if JoinType supports pushing predicates into its inputs

2023-09-06 Thread via GitHub
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:

[GitHub] [calcite] rubenada merged pull request #3403: [CALCITE-5967] UnsupportedOperationException while implementing a call that requires a special collator

2023-09-06 Thread via GitHub
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:

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3409: [CALCITE-5980] QuidemTests are not effectively executed on Windows

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3395: [CALCITE-5948] Explicit casting should be made if the type of an element in ARRAY/MAP not equals with the derived component type

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3403: [CALCITE-5967] UnsupportedOperationException while implementing a call that requires a special collator

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3407: [CALCITE-5952] SemiJoinJoinTransposeRule should check if JoinType supports pushing predicates into its inputs

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3403: [CALCITE-5967] UnsupportedOperationException while implementing a call that requires a special collator

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] rubenada opened a new pull request, #3409: [CALCITE-5980] QuidemTests are not effectively executed on Windows

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-06 Thread via GitHub
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

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

2023-09-04 Thread via GitHub
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) {

[GitHub] [calcite] kgyrtkirk merged pull request #3400: [CALCITE-5953] AggregateCaseToFilterRule may make inaccurate SUM transformations

2023-09-05 Thread via GitHub
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:

[GitHub] [calcite] rubenada commented on a diff in pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-05 Thread via GitHub
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) {

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

2023-09-04 Thread via GitHub
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) {

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

2023-09-04 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-05 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3397: [CALCITE-5836] Implement Rel2Sql for MERGE

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] JiajunBernoulli commented on pull request #3382: [CALCITE-5944] Add metadata for Sample

2023-09-06 Thread via GitHub
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

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

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3382: [CALCITE-5944] Add metadata for Sample

2023-09-06 Thread via GitHub
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

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

2023-09-06 Thread via GitHub
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

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

2023-09-06 Thread via GitHub
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

[GitHub] [calcite] LakeShen commented on a diff in pull request #3415: Avoid comparing root Relsubset's TraitSet with itself

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3397: [CALCITE-5836] Implement Rel2Sql for MERGE

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3397: [CALCITE-5836] Implement Rel2Sql for MERGE

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3417: [CALCITE-5993] Add CODE_POINTS_TO_STRING, TO_CODE_POINTS function (en…

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3406: [CALCITE-5974] Elasticsearch adapter throws ClassCastException when index mapping sets dynamic_templates without properties

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3415: Avoid comparing root Relsubset's TraitSet with itself

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3416: [CALCITE-4189] Simplify 'p OR (p IS NOT TRUE)' to 'TRUE'

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] LakeShen commented on pull request #3414: Reduce the number of tasks related to AbstractConverter in TopDownRuleDriver

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] asdfgh19 opened a new pull request, #3414: Reduce the number of tasks related to AbstractConverter in TopDownRuleDriver

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] macroguo-ghy opened a new pull request, #3416: [CALCITE-4189] Simplify 'p OR (p IS NOT TRUE)' to 'TRUE'

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3390: [CALCITE-5949] RexExecutable should return unchanged original expressions when it fails

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3416: [CALCITE-4189] Simplify 'p OR (p IS NOT TRUE)' to 'TRUE'

2023-09-10 Thread via GitHub
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

[GitHub] [calcite] macroguo-ghy opened a new pull request, #3417: [CALCITE-5993] Add CODE_POINTS_TO_STRING, TO_CODE_POINTS function (en…

2023-09-10 Thread via GitHub
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.

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3414: Reduce the number of tasks related to AbstractConverter in TopDownRuleDriver

2023-09-10 Thread via GitHub
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

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

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3410: [CALCITE-5971] Add the RelRule to rewrite the bernoulli sample as Filter

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3410: [CALCITE-5971] Add the RelRule to rewrite the bernoulli sample as Filter

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] chucheng92 commented on pull request #3405: [CALCITE-5961] Type inference of ARRAY_COMPACT is incorrect

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on pull request #3311: [CALCITE-5732] EnumerableHashJoin and EnumerableMergeJoin on composite key return rows matching condition 'null = null'

2023-09-07 Thread via GitHub
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,

[GitHub] [calcite] LakeShen commented on a diff in pull request #3404: [CALCITE-5968] Provide an interface for RexExecutable to decouple janino runtime binding

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] LakeShen commented on pull request #3406: [CALCITE-5974] Fix bug when es index mapping set dynamic_templates without properties

2023-09-07 Thread via GitHub
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

[GitHub] [calcite] sonarcloud[bot] commented on pull request #3416: [CALCITE-4189] Simplify 'p OR (p IS NOT TRUE)' to 'TRUE'

2023-09-11 Thread via GitHub
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

[GitHub] [calcite] rubenada commented on a diff in pull request #3390: [CALCITE-5949] RexExecutable should return unchanged original expressions when it fails

2023-09-11 Thread via GitHub
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

[GitHub] [calcite] macroguo-ghy commented on a diff in pull request #3416: [CALCITE-4189] Simplify 'p OR (p IS NOT TRUE)' to 'TRUE'

2023-09-11 Thread via GitHub
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) { //

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

2023-09-11 Thread via GitHub
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   2   3   4   5   6   7   8   9   10   >