You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by "xuzifu666 (via GitHub)" <gi...@apache.org> on 2026/08/09 03:41:29 UTC
[PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
xuzifu666 opened a new pull request, #5169:
URL: https://github.com/apache/calcite/pull/5169
jira: https://issues.apache.org/jira/browse/CALCITE-5413
This issue should had been fixed, so I only add a test case to cover it.
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "iwanttobepowerful (via GitHub)" <gi...@apache.org>.
iwanttobepowerful commented on PR #5169:
URL: https://github.com/apache/calcite/pull/5169#issuecomment-5229866075
add QuidemTest
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #5169:
URL: https://github.com/apache/calcite/pull/5169#discussion_r3743393555
##########
core/src/test/resources/sql/new-decorr.iq:
##########
@@ -496,4 +496,39 @@ SELECT empno, (SELECT row_number() OVER (PARTITION BY dname ORDER BY emp.sal) FR
!ok
!}
+
+# [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly
+# The innermost subquery correlates to both the outermost (e) and the middle (d) query levels.
Review Comment:
I had add QuidemTest to cover it and result is expected with PostgreSQL: https://onecompiler.com/postgresql/44xqduwb9
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #5169:
URL: https://github.com/apache/calcite/pull/5169#issuecomment-5235592825
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_duplicated_lines_density&view=list)
<!-- sqra-placement-anchor -->
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169)
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #5169:
URL: https://github.com/apache/calcite/pull/5169#issuecomment-5231001236
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_duplicated_lines_density&view=list)
<!-- sqra-placement-anchor -->
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169)
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 merged PR #5169:
URL: https://github.com/apache/calcite/pull/5169
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #5169:
URL: https://github.com/apache/calcite/pull/5169#issuecomment-5229685671
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=5169&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=5169&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=5169&metric=new_duplicated_lines_density&view=list)
<!-- sqra-placement-anchor -->
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=5169)
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Test cases for [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #5169:
URL: https://github.com/apache/calcite/pull/5169#discussion_r3743393555
##########
core/src/test/resources/sql/new-decorr.iq:
##########
@@ -496,4 +496,39 @@ SELECT empno, (SELECT row_number() OVER (PARTITION BY dname ORDER BY emp.sal) FR
!ok
!}
+
+# [CALCITE-5413] Nested Subqueries with correlated variables are not decorrelated correctly
+# The innermost subquery correlates to both the outermost (e) and the middle (d) query levels.
Review Comment:
I had add QuidemTest to cover it and result is expected with PostgreSQL: https://onecompiler.com/postgresql/44xqduwb9
cc @mihaibudiu @iwanttobepowerful
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org