mihaibudiu commented on code in PR #4361:
URL: https://github.com/apache/calcite/pull/4361#discussion_r2076436376
##########
core/src/test/resources/sql/planner.iq:
##########
@@ -132,12 +132,13 @@ EnumerableCalc(expr#0..2=[{inputs}], $f0=[$t1], $f1=[$t2])
# [CALCITE-7000] Extend IntersectToSemiJoinRule to support n-way inputs
!set planner-rules "-EnumerableRules.ENUMERABLE_INTERSECT_RULE,
+ -CoreRules.INTERSECT_TO_DISTINCT,
+CoreRules.INTERSECT_TO_SEMI_JOIN"
select a from (values (1), (2), (3), (4), (5)) as t1 (a)
intersect
select a from (values (1), (2)) as t2 (a)
intersect
-select a from (values (1), (4)) as t3 (a);
+select a from (values (1), (4), (null)) as t3 (a);
Review Comment:
I think the idea is to run the same test with both old and new rules and see
that the results are the same
(there is no way to say "the same", but the table output should be the same).
So can you also run it once with the existing rules?
--
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]
