This is an automated email from the ASF dual-hosted git repository.

morrysnow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new dda505487c [fix](nereids) SimplifyArithmeticRuleTest ut failed (#15486)
dda505487c is described below

commit dda505487c68f709e8f334a2cd1a19de08571df6
Author: minghong <engle...@gmail.com>
AuthorDate: Thu Dec 29 22:53:27 2022 +0800

    [fix](nereids) SimplifyArithmeticRuleTest ut failed (#15486)
    
    this PR remove typeCoercion on expected expr in 
ExpressionRewriteTestHelper. Because we should not rewrite expected expr at 
all. It will change the expected expr unexpectedly.
---
 .../nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java    | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java
 
b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java
index f4aabee6f4..75824b2441 100644
--- 
a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java
+++ 
b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java
@@ -66,7 +66,6 @@ public abstract class ExpressionRewriteTestHelper {
         Expression needRewriteExpression = PARSER.parseExpression(expression);
         needRewriteExpression = 
typeCoercion(replaceUnboundSlot(needRewriteExpression, mem));
         Expression expectedExpression = PARSER.parseExpression(expected);
-        expectedExpression = 
typeCoercion(replaceUnboundSlot(expectedExpression, mem));
         Expression rewrittenExpression = 
executor.rewrite(needRewriteExpression);
         Assertions.assertEquals(expectedExpression.toSql(), 
rewrittenExpression.toSql());
     }


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

Reply via email to