morrySnow commented on code in PR #44779:
URL: https://github.com/apache/doris/pull/44779#discussion_r1865699138


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/exploration/mv/HyperGraphComparator.java:
##########
@@ -75,9 +79,9 @@ public class HyperGraphComparator {
     private final Map<Edge, List<? extends Expression>> pullUpViewExprWithEdge 
= new HashMap<>();
     private final LogicalCompatibilityContext logicalCompatibilityContext;
     // this records the slots which needs to reject null
-    // the key is the target join which should reject null, the value is a 
pair, the first value of the pair is the
-    // join type, the second value is also a pair which left represents the 
slots in the left of join that should
-    // reject null, right represents the slots in the right of join that 
should reject null.
+    // the key is the view join edge which should reject null, the value is a 
pair, the first value of the pair is the

Review Comment:
   should not use view in Hyper comments because it could used in other scene



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to