(doris) branch branch-2.1 updated: branch-2.1: [fix](Nereids) fix fold constant of time acquired functions #47288 (#47920)

[morrysnow]

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

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


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new 3a747996078 branch-2.1: [fix](Nereids) fix fold constant of time 
acquired functions #47288 (#47920)
3a747996078 is described below

commit 3a747996078a4aa5c6958cbd4dcd15ee47208c4f
Author: LiBinfeng <libinf...@selectdb.com>
AuthorDate: Mon Feb 17 14:36:02 2025 +0800

    branch-2.1: [fix](Nereids) fix fold constant of time acquired functions 
#47288 (#47920)
    
    pick from master #47288
    
    Problem Summary:
    explain select substr(current_date, 1, 10);
    when logicalPlanBuilder build ast from original sql of date acquired
    functions like current_date, it would add an alias above. Which would
    stop folding constant when fold constant rule traversing expression tree
    So remove alias when translate to ast
---
 .../doris/nereids/parser/LogicalPlanBuilder.java   | 12 ++++----
 .../fold_constant_date_arithmatic.groovy           | 32 ++++++++++++++++++++++
 2 files changed, 38 insertions(+), 6 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
index 0ae1a209f55..b1f32c324e2 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
@@ -2017,32 +2017,32 @@ public class LogicalPlanBuilder extends 
DorisParserBaseVisitor<Object> {
 
     @Override
     public Expression visitCurrentDate(DorisParser.CurrentDateContext ctx) {
-        return new CurrentDate().alias("CURRENT_DATE");
+        return new CurrentDate();
     }
 
     @Override
     public Expression visitCurrentTime(DorisParser.CurrentTimeContext ctx) {
-        return new CurrentTime().alias("CURRENT_TIME");
+        return new CurrentTime();
     }
 
     @Override
     public Expression 
visitCurrentTimestamp(DorisParser.CurrentTimestampContext ctx) {
-        return new Now().alias("CURRENT_TIMESTAMP");
+        return new Now();
     }
 
     @Override
     public Expression visitLocalTime(DorisParser.LocalTimeContext ctx) {
-        return new CurrentTime().alias("LOCALTIME");
+        return new CurrentTime();
     }
 
     @Override
     public Expression visitLocalTimestamp(DorisParser.LocalTimestampContext 
ctx) {
-        return new Now().alias("LOCALTIMESTAMP");
+        return new Now();
     }
 
     @Override
     public Expression visitCurrentUser(DorisParser.CurrentUserContext ctx) {
-        return new CurrentUser().alias("CURRENT_USER");
+        return new CurrentUser();
     }
 
     @Override
diff --git 
a/regression-test/suites/nereids_p0/expression/fold_constant/fold_constant_date_arithmatic.groovy
 
b/regression-test/suites/nereids_p0/expression/fold_constant/fold_constant_date_arithmatic.groovy
new file mode 100644
index 00000000000..5e0ed2e5d25
--- /dev/null
+++ 
b/regression-test/suites/nereids_p0/expression/fold_constant/fold_constant_date_arithmatic.groovy
@@ -0,0 +1,32 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+suite("fold_constant_date_arithmatic") {
+    def db = "fold_constant_date_arithmatic"
+    sql "create database if not exists ${db}"
+
+    sql "set enable_nereids_planner=true"
+    sql "set enable_fallback_to_original_planner=false"
+    sql "set enable_fold_constant_by_be=false"
+
+    testFoldConst("select substr(now(), 1, 10);")
+    testFoldConst("select substr(now(3), 1, 10);")
+    testFoldConst("select substr(curdate(), 1, 10);")
+    testFoldConst("select substr(current_date(), 1, 10);")
+    testFoldConst("select substr(current_timestamp(), 1, 10);")
+    testFoldConst("select substr(current_timestamp(3), 1, 10);")
+}


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