xinyiZzz commented on a change in pull request #8745:
URL: https://github.com/apache/incubator-doris/pull/8745#discussion_r839556206
##########
File path: fe/fe-core/src/main/java/org/apache/doris/planner/Planner.java
##########
@@ -266,6 +271,57 @@ public void createPlanFragments(StatementBase statement,
Analyzer analyzer, TQue
}
}
+ private class FindTuplePredicate implements
com.google.common.base.Predicate<PlanNode> {
+ private final TupleId tupleId;
+
+ FindTuplePredicate(TupleId id) {
+ tupleId = id;
+ }
+
+ @Override
+ public boolean apply(PlanNode arg) {
+ return arg.getTupleIds().size() == 1 && arg.getTupleIds().get(0)
== tupleId
Review comment:
I understand that filtering conjuncts to be empty is to avoid invalid
filtering in some primary-foreign key joins, because the key values on the
left and right sides are the same at this time; limiting the number of Tuples
to 1 skips the child join node.
It is recommended to add a note here, it is not very well understood;
--
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]
