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

Fix bug: the result of interval function is wrong.

Browse Source
This commit is contained in:
obdev
2023-08-01 14:18:29 +00:00
committed by ob-robot
parent f47722e7ac
commit 23e086d49d
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/sql/engine/expr/ob_expr_interval.cpp
View File

@ -122,23 +122,21 @@ int ObExprInterval::calc_interval_expr(const ObExpr &expr, ObEvalCtx &ctx,
} else { } else {
int64_t high = expr.arg_cnt_ - 1; int64_t high = expr.arg_cnt_ - 1;
int64_t low = 1; int64_t low = 1;
int64_t mid = 0;
while (low <= high && OB_SUCC(ret)) { while (low <= high && OB_SUCC(ret)) {
int64_t mid = (low + high) / 2;
const ObDatum &arg_i = expr.locate_param_datum(ctx, static_cast<int>(mid)); const ObDatum &arg_i = expr.locate_param_datum(ctx, static_cast<int>(mid));
mid = (low + high + 1) / 2;
if (OB_FAIL(cmp_func(arg_i, *arg0, cmp_ret))) { if (OB_FAIL(cmp_func(arg_i, *arg0, cmp_ret))) {
LOG_WARN("faile to compare", K(ret)); LOG_WARN("faile to compare", K(ret));
} else if (cmp_ret > 0) { } else if (cmp_ret <= 0) {
high = mid - 1; low = mid + 1;
mid -= 1;
} else if (low == high) { } else if (low == high) {
break; break;
} else { } else {
low = mid; high = mid - 1;
} }
} // while } // while
if (OB_SUCC(ret)) { if (OB_SUCC(ret)) {
res.set_int(mid); res.set_int(low - 1);
} }
} }
} }