[BugFix] Fix Expr::open crash when evaluate const throw exception (#5…

…2752)

Signed-off-by: stdpain <drfeng08@gmail.com>
(cherry picked from commit 130facd)

be/src/exprs/expr_context.cpp

@@ -84,7 +84,11 @@ Status ExprContext::open(RuntimeState* state) {
     // original's fragment state and only need to have thread-local state initialized.
     FunctionContext::FunctionStateScope scope =
             _is_clone ? FunctionContext::THREAD_LOCAL : FunctionContext::FRAGMENT_LOCAL;
-    return _root->open(state, this, scope);
+    try {
+        return _root->open(state, this, scope);
+    } catch (std::runtime_error& e) {
+        return Status::RuntimeError(fmt::format("Expr evaluate meet error: {}", e.what()));
+    }
 }
 
 Status ExprContext::open(std::vector<ExprContext*> evals, RuntimeState* state) {

test/sql/test_exception/R/test_number_overflow

@@ -0,0 +1,12 @@
+-- name: test_number_overflow
+set sql_mode="ERROR_IF_OVERFLOW";
+-- result:
+-- !result
+select cast(abs(1234567890123456789) as decimal(4,3));
+-- result:
+[REGEX].*Expr evaluate meet error: The type cast from other types to decimal overflows.*
+-- !result
+select 1 in (cast(abs(1234567890123456789) as decimal(4,3)), cast(abs(1234567890123456789) as decimal(4,3)));
+-- result:
+[REGEX].*Expr evaluate meet error: The type cast from other types to decimal overflows.*
+-- !result

test/sql/test_exception/T/test_number_overflow

@@ -0,0 +1,7 @@
+-- name: test_number_overflow
+set sql_mode="ERROR_IF_OVERFLOW";
+
+-- exception in evaluate
+select cast(abs(1234567890123456789) as decimal(4,3));
+-- exception in open
+select 1 in (cast(abs(1234567890123456789) as decimal(4,3)), cast(abs(1234567890123456789) as decimal(4,3)));