This is an automated email from the ASF dual-hosted git repository.
yiguolei 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 ace2b45c377 [fix](decimalv2) avoid crashing caused by decimalv2
arithmetic with check_overflow_for_decimal enabled (#28219)
ace2b45c377 is described below
commit ace2b45c377fb72338babf1415a96850458862b8
Author: Jerry Hu <[email protected]>
AuthorDate: Tue Dec 12 20:32:10 2023 +0800
[fix](decimalv2) avoid crashing caused by decimalv2 arithmetic with
check_overflow_for_decimal enabled (#28219)
---
be/src/vec/functions/function_binary_arithmetic.h | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/be/src/vec/functions/function_binary_arithmetic.h
b/be/src/vec/functions/function_binary_arithmetic.h
index ab89c3f8543..a58417b71e2 100644
--- a/be/src/vec/functions/function_binary_arithmetic.h
+++ b/be/src/vec/functions/function_binary_arithmetic.h
@@ -961,6 +961,8 @@ public:
}
bool check_overflow_for_decimal =
context->check_overflow_for_decimal();
+ auto status = Status::RuntimeError("{}'s arguments do not match the
expected data types",
+ get_name());
bool valid = cast_both_types(
left_generic, right_generic, result_generic,
[&](const auto& left, const auto& right, const auto& res) {
@@ -978,6 +980,13 @@ public:
(IsDataTypeDecimal<LeftDataType> ||
IsDataTypeDecimal<RightDataType>))) {
if (check_overflow_for_decimal) {
+ if constexpr ((IsDecimalV2<typename
LeftDataType::FieldType> ||
+ IsDecimalV2<typename RightDataType::
+
FieldType>)&&!is_to_null_type) {
+ status =
Status::Error<ErrorCode::NOT_IMPLEMENTED_ERROR>(
+ "cannot check overflow with
decimalv2");
+ return false;
+ }
auto column_result = ConstOrVectorAdapter<
LeftDataType, RightDataType,
std::conditional_t<IsDataTypeDecimal<ExpectedResultDataType>,
@@ -1007,8 +1016,7 @@ public:
return false;
});
if (!valid) {
- return Status::RuntimeError("{}'s arguments do not match the
expected data types",
- get_name());
+ return status;
}
return Status::OK();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
