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


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/ExpressionNormalization.java:
##########
@@ -67,7 +68,10 @@ public class ExpressionNormalization extends 
ExpressionRewrite {
                 LogToLn.INSTANCE,
                 ConcatWsMultiArrayToOne.INSTANCE,
                 TimestampToAddTime.INSTANCE,
-                FoldConstantRule.INSTANCE,
+                FoldConstantRule.INSTANCE
+            ),
+            NullIfToIf.INSTANCE,

Review Comment:
   如果要做这个改写,我建议在plan builder的时候就搞。这里一直想这么改,还没来得及动



-- 
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]

Reply via email to