seawinde commented on code in PR #61345:
URL: https://github.com/apache/doris/pull/61345#discussion_r3050999778
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rules/AddMinMax.java:
##########
@@ -185,22 +185,22 @@ private Expression addExprMinMaxValues(Expression expr,
ExpressionRewriteContext
&& range.lowerEndpoint().equals(range.upperEndpoint())
&& range.lowerBoundType() == BoundType.CLOSED
&& range.upperBoundType() == BoundType.CLOSED) {
- Expression cmp = new EqualTo(targetExpr, (Literal)
range.lowerEndpoint());
+ Expression cmp = new EqualTo(targetExpr, (Literal)
range.lowerEndpoint()).withInferred(true);
Review Comment:
a test for the inferred-predicate path, to verify that predicates generated
by AddMinMax or InferPredicates do not participate in residual compensation.
This matters because the
follow-up fix relies on filtering inferred predicates during candidate
collection.
--
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]
