rubenada commented on code in PR #3675:
URL: https://github.com/apache/calcite/pull/3675#discussion_r1482547365
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -1799,11 +1801,16 @@ private boolean innerHasNext() {
i = -1;
JiajunBernoulli commented on code in PR #3675:
URL: https://github.com/apache/calcite/pull/3675#discussion_r1482360787
##
linq4j/src/main/java/org/apache/calcite/linq4j/EnumerableDefaults.java:
##
@@ -1799,11 +1801,16 @@ private boolean innerHasNext() {
i = -1;
sonarcloud[bot] commented on PR #3641:
URL: https://github.com/apache/calcite/pull/3641#issuecomment-1933291861
## [])
JiajunBernoulli commented on code in PR #3641:
URL: https://github.com/apache/calcite/pull/3641#discussion_r1482347529
##
core/src/test/resources/org/apache/calcite/test/SqlToRelConverterTest.xml:
##
@@ -6343,6 +6343,161 @@ LogicalAggregate(group=[{0}], CNT=[COUNT()])
sonarcloud[bot] commented on PR #3672:
URL: https://github.com/apache/calcite/pull/3672#issuecomment-1933263478
## [])
JiajunBernoulli commented on code in PR #3641:
URL: https://github.com/apache/calcite/pull/3641#discussion_r1482328835
##
core/src/test/resources/org/apache/calcite/test/SqlToRelConverterTest.xml:
##
@@ -6343,6 +6343,161 @@ LogicalAggregate(group=[{0}], CNT=[COUNT()])
mihaibudiu commented on code in PR #3672:
URL: https://github.com/apache/calcite/pull/3672#discussion_r1482322829
##
core/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java:
##
@@ -1802,11 +1802,11 @@ public class SqlStdOperatorTable extends
JiajunBernoulli commented on code in PR #3674:
URL: https://github.com/apache/calcite/pull/3674#discussion_r1482320473
##
core/src/main/java/org/apache/calcite/adapter/enumerable/EnumerableBatchNestedLoopJoin.java:
##
@@ -124,8 +124,8 @@ public static
This is an automated email from the ASF dual-hosted git repository.
mbudiu 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 f837ffa877 [CALCITE-6228] ELEMENT function infers
mihaibudiu merged PR #3650:
URL: https://github.com/apache/calcite/pull/3650
--
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:
HanumathRao commented on PR #3640:
URL: https://github.com/apache/calcite/pull/3640#issuecomment-1932905283
> @HanumathRao could you please elaborate why we need in this PR the changes
in `CassandraSchema.java` and `Prepare.java`? Is is related to the
decorrelation problem, or could they
sonarcloud[bot] commented on PR #3650:
URL: https://github.com/apache/calcite/pull/3650#issuecomment-1932806091
## [ {
public static
mihaibudiu commented on PR #3674:
URL: https://github.com/apache/calcite/pull/3674#issuecomment-1932740859
This looks fine, but no test is affected, so it's hard to gauge the actual
impact of this change.
--
This is an automated message from the Apache Git Service.
To respond to the
mihaibudiu commented on PR #3676:
URL: https://github.com/apache/calcite/pull/3676#issuecomment-1932637903
The rule is for the PR name, the JIRA issue, and the commit message to be
the same.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
rubenada commented on PR #3640:
URL: https://github.com/apache/calcite/pull/3640#issuecomment-1932326838
@HanumathRao could you please elaborate why we need in this PR the changes
in `CassandraSchema.java` and `Prepare.java`? Is is related to the
decorrelation problem, or could they be
rubenada commented on PR #3675:
URL: https://github.com/apache/calcite/pull/3675#issuecomment-1932186954
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 #3676:
URL: https://github.com/apache/calcite/pull/3676#issuecomment-1932129472
## [
--
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,
sonarcloud[bot] commented on PR #3668:
URL: https://github.com/apache/calcite/pull/3668#issuecomment-1931862374
## [ {
public static
rubenada commented on PR #3674:
URL: https://github.com/apache/calcite/pull/3674#issuecomment-1931738219
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
YiwenWu commented on code in PR #3650:
URL: https://github.com/apache/calcite/pull/3650#discussion_r1481239292
##
core/src/main/java/org/apache/calcite/sql/type/ReturnTypes.java:
##
@@ -620,6 +620,13 @@ public static SqlCall stripSeparator(SqlCall call) {
public static final
sonarcloud[bot] commented on PR #3674:
URL: https://github.com/apache/calcite/pull/3674#issuecomment-1931597625
## [ were added to refs/heads/main by this push:
new f7069cc524 [CALCITE-5647] RelMdPopulationSize
rubenada merged PR #3632:
URL: https://github.com/apache/calcite/pull/3632
--
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 #3648:
URL: https://github.com/apache/calcite/pull/3648#issuecomment-1931497457
## [