database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[fix](nereids) SimplifyArithmeticRuleTest ut failed (#15486)

Browse Source 
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.
This commit is contained in:
minghong
2022-12-29 22:53:27 +08:00
committed by GitHub
parent bb305aa572
commit dda505487c
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
fe/fe-core/src/test/java/org/apache/doris/nereids/rules/expression/rewrite/ExpressionRewriteTestHelper.java
View File

@ -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());
}