seawinde commented on code in PR #66182:
URL: https://github.com/apache/doris/pull/66182#discussion_r3679558797
##########
regression-test/suites/query_p0/hint/fix_leading.groovy:
##########
@@ -193,8 +193,9 @@ suite("fix_leading") {
// check left right join result
qt_select4_1 """select count(*) from t1 left join t2 on c1 > 500 and c2
>500 right join t3 on c3 > 500 and c1 < 200;"""
- qt_select4_2 """select /*+ leading(t1 t2 t3)*/ count(*) from t1 left join
t2 on c1 > 500 and c2 >500 right join t3 on c3 > 500 and c1 < 200;"""
- qt_select4_3 """explain shape plan select /*+ leading(t1 t2 t3)*/ count(*)
from t1 left join t2 on c1 > 500 and c2 >500 right join t3 on c3 > 500 and c1 <
200;"""
+ // TODO: Re-enable after LEADING handles right join constraints without
Analyzer normalization.
Review Comment:
EADING encode right-join constraints symmetrically (the same concern applies
to preserved-side RIGHT ANTI predicates) would be in another pr
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
