libenchao commented on code in PR #3193:
URL: https://github.com/apache/calcite/pull/3193#discussion_r1207623563
##########
core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java:
##########
@@ -6823,6 +6823,48 @@ private void checkSemiJoinRuleOnAntiJoin(RelOptRule
rule) {
.checkUnchanged();
}
+ /** Test case for CALCITE-5683 for two level nested decorrelate with
standard program
+ * failing during the decorrelation phase. */
+ @Test void testTwoLevelDecorrelate() {
+ final String sql = "SELECT d1.name, d1.deptno + "
+ + " ( SELECT e1.empno "
+ + " FROM emp e1 "
+ + " WHERE d1.deptno = e1.deptno and "
+ + " e1.sal = (SELECT max(sal) "
+ + " FROM emp e2 "
+ + " WHERE e1.sal = e2.sal and"
+ + " e1.deptno = e2.deptno and"
+ + " d1.deptno < e2.deptno))"
+ + " FROM dept d1";
+
+ sql(sql)
+ .withExpand(false)
+ .withLateDecorrelate(true)
+ .withTrim(true)
+ .withRule()
+ .checkUnchanged();
Review Comment:
I think it's worth to keep these tests, my question is about why
`checkUnchanged` test contains `planAfter`, I'll check this and give my further
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, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
