expression: fail ColumnSubstituteImpl if creating function returns error (#53716) (#53815)

close #53580, close #53582, close #53594, close #53603

Author: ti-chi-bot

expression/integration_test.go

@@ -8064,3 +8064,17 @@ func TestCastBinaryStringToJSON(t *testing.T) {
 	tk.MustQuery("select 1 from t where cast(BINARY vc as json) = '1';").Check(testkit.Rows())
 	tk.MustQuery("select 1 from t where cast(BINARY c as json) = '1';").Check(testkit.Rows())
 }
+
+func TestTestIssue53580(t *testing.T) {
+	store := testkit.CreateMockStore(t)
+	tk := testkit.NewTestKit(t, store)
+
+	tk.MustExec("use test")
+	tk.MustExec("create table t (col TEXT);")
+	tk.MustQuery(`
+	select 1 from (select t.col as c0, 46578369 as c1 from t) as t where
+	case when (
+	  t.c0 in (t.c0, cast((cast(1 as unsigned) - cast(t.c1 as signed)) as char))
+	) then 1 else 2 end;
+	`).Check(testkit.Rows())
+}

expression/util.go

@@ -487,7 +487,11 @@ func ColumnSubstituteImpl(expr Expression, schema *Schema, newExprs []Expression
 			}
 		}
 		if substituted {
-			return true, hasFail, NewFunctionInternal(v.GetCtx(), v.FuncName.L, v.RetType, refExprArr.Result()...)
+			newFunc, err := NewFunction(v.GetCtx(), v.FuncName.L, v.RetType, refExprArr.Result()...)
+			if err != nil {
+				return true, true, v
+			}
+			return true, hasFail, newFunc
 		}
 	}
 	return false, false, expr