diff --git a/src/sql/engine/expr/ob_batch_eval_util.h b/src/sql/engine/expr/ob_batch_eval_util.h index 9996d1cedb..11971eac64 100644 --- a/src/sql/engine/expr/ob_batch_eval_util.h +++ b/src/sql/engine/expr/ob_batch_eval_util.h @@ -615,9 +615,11 @@ int def_fixed_len_vector_arith_op(VECTOR_EVAL_FUNC_ARG_DECL, Args &... args) VEC_ARG_LIST, args...); } } - SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, pvt_bound))); - SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, pvt_bound))); - SQL_LOG(DEBUG, "expr.args_[1]", K(ToStrVectorHeader(expr.args_[1]->get_vector_header(ctx), *expr.args_[1], &skip, pvt_bound))); + if (OB_SUCC(ret)) { + SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, pvt_bound))); + SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, pvt_bound))); + SQL_LOG(DEBUG, "expr.args_[1]", K(ToStrVectorHeader(expr.args_[1]->get_vector_header(ctx), *expr.args_[1], &skip, pvt_bound))); + } return ret; } @@ -658,9 +660,11 @@ int def_variable_len_vector_arith_op(VECTOR_EVAL_FUNC_ARG_DECL, Args &... args) VEC_ARG_LIST, args...); } } - SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, pvt_bound))); - SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, pvt_bound))); - SQL_LOG(DEBUG, "expr.args_[1]", K(ToStrVectorHeader(expr.args_[1]->get_vector_header(ctx), *expr.args_[1], &skip, pvt_bound))); + if (OB_SUCC(ret)) { + SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, pvt_bound))); + SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, pvt_bound))); + SQL_LOG(DEBUG, "expr.args_[1]", K(ToStrVectorHeader(expr.args_[1]->get_vector_header(ctx), *expr.args_[1], &skip, pvt_bound))); + } return ret; } #undef FORMAT_DISPATCH_BRANCH diff --git a/src/sql/engine/expr/ob_expr_substr.cpp b/src/sql/engine/expr/ob_expr_substr.cpp index 3c8b386fdc..7eae0fdf97 100644 --- a/src/sql/engine/expr/ob_expr_substr.cpp +++ b/src/sql/engine/expr/ob_expr_substr.cpp @@ -997,8 +997,10 @@ int ObExprSubstr::eval_substr_vector(VECTOR_EVAL_FUNC_ARG_DECL) ret = vector_substr(VECTOR_EVAL_FUNC_ARG_LIST); } } - SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, bound))); - SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, bound))); + if (OB_SUCC(ret)) { + SQL_LOG(DEBUG, "expr", K(ToStrVectorHeader(expr.get_vector_header(ctx), expr, &skip, bound))); + SQL_LOG(DEBUG, "expr.args_[0]", K(ToStrVectorHeader(expr.args_[0]->get_vector_header(ctx), *expr.args_[0], &skip, bound))); + } return ret; } DEF_SET_LOCAL_SESSION_VARS(ObExprSubstr, raw_expr) {