You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2023/02/08 02:51:42 UTC

[doris] branch master updated: [fix](nereids) FE nereids use DateV2Literal instead of 'cast datev2' (#16386)

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 254790c564 [fix](nereids) FE nereids use DateV2Literal instead of 'cast datev2' (#16386)
254790c564 is described below

commit 254790c564846b710e57dd836dd1cf5c2ea486fe
Author: minghong <en...@gmail.com>
AuthorDate: Wed Feb 8 10:51:35 2023 +0800

    [fix](nereids) FE nereids use DateV2Literal instead of 'cast datev2' (#16386)
    
    BE already support DateV2Literal, and hence, remove code in FE which convert DateV2Literal to Cast datev2
---
 .../doris/nereids/glue/translator/ExpressionTranslator.java   | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/ExpressionTranslator.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/ExpressionTranslator.java
index 38de2b8b96..776d02a3a8 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/ExpressionTranslator.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/glue/translator/ExpressionTranslator.java
@@ -76,10 +76,8 @@ import org.apache.doris.nereids.trees.expressions.functions.generator.TableGener
 import org.apache.doris.nereids.trees.expressions.functions.scalar.JsonArray;
 import org.apache.doris.nereids.trees.expressions.functions.scalar.JsonObject;
 import org.apache.doris.nereids.trees.expressions.functions.scalar.ScalarFunction;
-import org.apache.doris.nereids.trees.expressions.literal.DateLiteral;
 import org.apache.doris.nereids.trees.expressions.literal.DateTimeLiteral;
 import org.apache.doris.nereids.trees.expressions.literal.DateTimeV2Literal;
-import org.apache.doris.nereids.trees.expressions.literal.DateV2Literal;
 import org.apache.doris.nereids.trees.expressions.literal.Literal;
 import org.apache.doris.nereids.trees.expressions.literal.NullLiteral;
 import org.apache.doris.nereids.trees.expressions.visitor.DefaultExpressionVisitor;
@@ -211,15 +209,6 @@ public class ExpressionTranslator extends DefaultExpressionVisitor<Expr, PlanTra
         return nullLit;
     }
 
-    @Override
-    public Expr visitDateLiteral(DateLiteral dateLiteral, PlanTranslatorContext context) {
-        // BE not support date v2 literal and datetime v2 literal
-        if (dateLiteral instanceof DateV2Literal) {
-            return new CastExpr(Type.DATEV2, new StringLiteral(dateLiteral.toString()));
-        }
-        return super.visitDateLiteral(dateLiteral, context);
-    }
-
     @Override
     public Expr visitDateTimeLiteral(DateTimeLiteral dateTimeLiteral, PlanTranslatorContext context) {
         // BE not support date v2 literal and datetime v2 literal


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