hcq/oceanbase
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Correct the method of determining whether there are nulls on the right side in the IN expression.

Browse Source
This commit is contained in:
obdev
2023-12-27 08:43:11 +00:00
committed by ob-robot
parent 36129f954c
commit a25e14fe74
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/sql/engine/expr/ob_expr_in.cpp
View File

@ -1681,13 +1681,9 @@ int ObExprInOrNotIn::inner_eval_vector_in_without_row(const ObExpr &expr,
for (int i = 0; i < right_param_num; i++) { for (int i = 0; i < right_param_num; i++) {
in_ctx->cmp_functions_[i] = (void *)func_ptr; in_ctx->cmp_functions_[i] = (void *)func_ptr;
} }
if (std::is_same<LeftVec, ObFixedLengthBase>::value) {
fixed_base_l_payload = (reinterpret_cast<ObFixedLengthBase *>(input_left_vec))->get_data();
left_datum.len_ = (reinterpret_cast<ObFixedLengthBase *>(input_left_vec))->get_length();
}
if (0 == in_ctx->get_static_engine_hashset_size()) { if (0 == in_ctx->get_static_engine_hashset_size()) {
// Scenarios where in_list contains only null. // Scenarios where in_list contains only null.
if (right_has_null) { if (in_ctx->ctx_hash_null_) {
for (int64_t left_idx = bound.start(); left_idx < bound.end(); ++left_idx) { for (int64_t left_idx = bound.start(); left_idx < bound.end(); ++left_idx) {
if (skip.at(left_idx) || eval_flags.at(left_idx)) { continue; } if (skip.at(left_idx) || eval_flags.at(left_idx)) { continue; }
res_vec->set_null(left_idx); res_vec->set_null(left_idx);
@ -1699,6 +1695,9 @@ int ObExprInOrNotIn::inner_eval_vector_in_without_row(const ObExpr &expr,
LOG_WARN("static_engine_hashset_size unexpected", K(ret), K(right_has_null), LOG_WARN("static_engine_hashset_size unexpected", K(ret), K(right_has_null),
K(in_ctx->get_static_engine_hashset_size())); K(in_ctx->get_static_engine_hashset_size()));
} }
} else if (std::is_same<LeftVec, ObFixedLengthBase>::value) {
fixed_base_l_payload = (reinterpret_cast<ObFixedLengthBase *>(input_left_vec))->get_data();
left_datum.len_ = (reinterpret_cast<ObFixedLengthBase *>(input_left_vec))->get_length();
} }
} }
} }