michalstutzmann commented on code in PR #26895:
URL: https://github.com/apache/flink/pull/26895#discussion_r2286216744
##########
flink-table/flink-table-planner/src/test/scala/org/apache/flink/table/planner/plan/rules/logical/subquery/SubQuerySemiJoinTest.scala:
##########
@@ -1595,10 +1595,7 @@ class SubQuerySemiJoinTest extends SubQueryTestBase {
//
// LogicalFilter lost variablesSet information.
- util.verifyRelPlanNotExpected(sqlQuery1, "joinType=[semi]")
-
- val sqlQuery2 = "SELECT MAX(a) FROM x GROUP BY 1 HAVING EXISTS (SELECT 1
FROM y WHERE d < b)"
- util.verifyRelPlanNotExpected(sqlQuery2, "joinType=[semi]")
+ util.verifyRelPlanNotExpected(sqlQuery, "joinType=[semi]")
Review Comment:
Related to a bug fix
[CALCITE-6032](https://issues.apache.org/jira/browse/CALCITE-6032),
specifically [this
line](https://github.com/apache/calcite/blame/calcite-1.37.0/core/src/main/java/org/apache/calcite/sql2rel/SqlToRelConverter.java#L5526).
After the change the query `sqlQuery2` no longer works and throws "'x.b' is
not a group expression".
I assume the intention of the test `testExistsWithCorrelatedOnHaving` is
covered already by the the other query `sqlQuery1`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
