You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ya...@apache.org on 2022/12/21 07:38:12 UTC
[doris] branch branch-1.2-lts updated: [Bugfix](round) fix round function may coredump (#15235)
This is an automated email from the ASF dual-hosted git repository.
yangzhg pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
new a6f5ec9efb [Bugfix](round) fix round function may coredump (#15235)
a6f5ec9efb is described below
commit a6f5ec9efb997d1cdd56c050fc15300dd6a3cb70
Author: Zhengguo Yang <ya...@gmail.com>
AuthorDate: Wed Dec 21 15:38:07 2022 +0800
[Bugfix](round) fix round function may coredump (#15235)
---
be/src/exprs/scalar_fn_call.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/be/src/exprs/scalar_fn_call.cpp b/be/src/exprs/scalar_fn_call.cpp
index 7212b619bb..b023df6ff5 100644
--- a/be/src/exprs/scalar_fn_call.cpp
+++ b/be/src/exprs/scalar_fn_call.cpp
@@ -151,7 +151,8 @@ Status ScalarFnCall::open(RuntimeState* state, ExprContext* ctx,
// determines how many decimal places are printed.
// TODO: revisit this. We should be able to do this if the scale argument is
// non-constant.
- if (_fn.name.function_name == "round" && _type.type == TYPE_DOUBLE) {
+ if (_fn.name.function_name == "round" && _fn.__isset.signature &&
+ _fn.signature.find("round_up_to") != std::string::npos) {
DCHECK_EQ(_children.size(), 2);
if (_children[1]->is_constant()) {
IntVal scale_arg = _children[1]->get_int_val(ctx, nullptr);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org