update

Signed-off-by: Weizhen Wang <[email protected]>

Author: hawkingrei

pkg/planner/core/constraint/exprs.go

@@ -44,13 +44,13 @@ func DeleteTrueExprs(p base.LogicalPlan, conds []expression.Expression) []expres
 
 // IsConstFalse is used to check whether the expression is a constant false expression.
 func IsConstFalse(ctx expression.BuildContext, expr expression.Expression) bool {
-	if expression.MaybeOverOptimized4PlanCache(ctx, []expression.Expression{expr}) {
-		ctx.SetSkipPlanCache("some parameters may be overwritten when constant propagation")
-	}
 	if e, ok := expr.(*expression.ScalarFunction); ok {
 		switch e.FuncName.L {
 		case ast.LT, ast.LE, ast.GT, ast.GE, ast.EQ, ast.NE:
 			if constExpr, ok := e.GetArgs()[1].(*expression.Constant); ok && constExpr.Value.IsNull() && constExpr.DeferredExpr == nil {
+				if expression.MaybeOverOptimized4PlanCache(ctx, []expression.Expression{expr}) {
+					ctx.SetSkipPlanCache("some parameters may be overwritten when constant propagation")
+				}
 				return true
 			}
 		}