From ac85fb8fa70b8289c77651132ab1348fa0b59389 Mon Sep 17 00:00:00 2001
From: TCeason <tai_chong@foxmail.com>
Date: Thu, 26 Dec 2024 10:53:28 +0800
Subject: [PATCH] fix(query): Disabled the shrink_scalar optimization for cast
 expressions

---
 .../sql/src/planner/semantic/type_check.rs    | 12 +++----
 .../base/15_procedure/15_0002_procedure.test  | 31 +++++++++++++++++++
 2 files changed, 37 insertions(+), 6 deletions(-)

diff --git a/src/query/sql/src/planner/semantic/type_check.rs b/src/query/sql/src/planner/semantic/type_check.rs
index ae48c35f55a9..f9ff56c44639 100644
--- a/src/query/sql/src/planner/semantic/type_check.rs
+++ b/src/query/sql/src/planner/semantic/type_check.rs
@@ -594,7 +594,7 @@ impl<'a> TypeChecker<'a> {
                 let registry = &BUILTIN_FUNCTIONS;
                 let checked_expr = type_check::check(&raw_expr, registry)?;
 
-                if let Some(constant) = self.try_fold_constant(&checked_expr) {
+                if let Some(constant) = self.try_fold_constant(&checked_expr, false) {
                     return Ok(constant);
                 }
                 // if the source type is nullable, cast target type should also be nullable.
@@ -637,7 +637,7 @@ impl<'a> TypeChecker<'a> {
                 let registry = &BUILTIN_FUNCTIONS;
                 let checked_expr = type_check::check(&raw_expr, registry)?;
 
-                if let Some(constant) = self.try_fold_constant(&checked_expr) {
+                if let Some(constant) = self.try_fold_constant(&checked_expr, false) {
                     return Ok(constant);
                 }
 
@@ -1086,7 +1086,6 @@ impl<'a> TypeChecker<'a> {
 
             Expr::Hole { .. } => unreachable!("hole is impossible in trivial query"),
         };
-
         Ok(Box::new((scalar, data_type)))
     }
 
@@ -2749,7 +2748,7 @@ impl<'a> TypeChecker<'a> {
             } => {
                 let mut folded_args = Vec::with_capacity(args.len());
                 for (checked_arg, arg) in checked_args.iter().zip(args.iter()) {
-                    match self.try_fold_constant(checked_arg) {
+                    match self.try_fold_constant(checked_arg, true) {
                         Some(constant) if arg.evaluable() => {
                             folded_args.push(constant.0);
                         }
@@ -2767,7 +2766,7 @@ impl<'a> TypeChecker<'a> {
             self.ctx.set_cacheable(false);
         }
 
-        if let Some(constant) = self.try_fold_constant(&expr) {
+        if let Some(constant) = self.try_fold_constant(&expr, true) {
             return Ok(constant);
         }
 
@@ -5067,8 +5066,9 @@ impl<'a> TypeChecker<'a> {
     fn try_fold_constant<Index: ColumnIndex>(
         &self,
         expr: &EExpr<Index>,
+        enable_shrink: bool,
     ) -> Option<Box<(ScalarExpr, DataType)>> {
-        if expr.is_deterministic(&BUILTIN_FUNCTIONS) {
+        if expr.is_deterministic(&BUILTIN_FUNCTIONS) && enable_shrink {
             if let (EExpr::Constant { scalar, .. }, _) =
                 ConstantFolder::fold(expr, &self.func_ctx, &BUILTIN_FUNCTIONS)
             {
diff --git a/tests/sqllogictests/suites/base/15_procedure/15_0002_procedure.test b/tests/sqllogictests/suites/base/15_procedure/15_0002_procedure.test
index 1d0ad55352bc..1142ef478a61 100644
--- a/tests/sqllogictests/suites/base/15_procedure/15_0002_procedure.test
+++ b/tests/sqllogictests/suites/base/15_procedure/15_0002_procedure.test
@@ -124,5 +124,36 @@ drop procedure if exists not_exists_p();
 statement error 3130
 drop procedure not_exists_p();
 
+statement ok
+drop procedure if exists sum_even_numbers(Int, Int);
+
+statement ok
+CREATE PROCEDURE sum_even_numbers(start_val Int, end_val Int)
+RETURNS UInt8 NOT NULL
+LANGUAGE SQL
+COMMENT='Calculate the sum of all even numbers'
+AS $$
+BEGIN
+    LET sum := 0;
+    FOR i IN start_val TO end_val DO
+        IF i % 2 = 0 THEN
+            sum := sum + i;
+        END IF;
+    END FOR;
+    RETURN sum;
+END;
+$$;
+
+statement error 3130
+call procedure sum_even_numbers(1, 2)
+
+query T
+call procedure sum_even_numbers(1::INT, 2::INT)
+----
+2
+
+statement ok
+drop procedure sum_even_numbers(Int, Int);
+
 statement ok
 unset global enable_experimental_procedure;