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

bugfix resolve between node when types are inconsistent

Browse Source
This commit is contained in:
obdev
2023-08-02 10:42:47 +00:00
committed by ob-robot
parent bb7ce82bd2
commit 7de309dcc7
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/sql/resolver/expr/ob_raw_expr_resolver_impl.cpp
View File

@ -3297,15 +3297,6 @@ int ObRawExprResolverImpl::process_between_node(const ParseNode *node, ObRawExpr
LOG_WARN("unexpected null", K(ret), K(i));
}
}
// The content of the 4th raw expr is same to that of the 1st raw expr.
// But the ptr addresses need to be different because our optimizer relys on it.
if (OB_FAIL(ret)) {
} else if (OB_FAIL(recursive_resolve(node->children_[0], btw_params[BTW_PARAM_NUM]))) {
SQL_RESV_LOG(WARN, "resolve child expr failed", K(ret), K(BTW_PARAM_NUM));
} else if (OB_ISNULL(btw_params[BTW_PARAM_NUM])) {
ret = OB_ERR_UNEXPECTED;
LOG_WARN("unexpected null", K(ret), K(BTW_PARAM_NUM));
}
if (OB_SUCC(ret) && lib::is_mysql_mode()) {
if (OB_ISNULL(ctx_.session_info_)) {
ret = OB_ERR_UNEXPECTED;
@ -3331,6 +3322,19 @@ int ObRawExprResolverImpl::process_between_node(const ParseNode *node, ObRawExpr
}
}
}
// The content of the 4th raw expr is same to that of the 1st raw expr.
// But the ptr addresses need to be different because our optimizer relys on it.
if (OB_SUCC(ret)) {
if (lib::is_mysql_mode() && !can_transform_in_mysql_mode) {
// do nothing
} else {
if (OB_FAIL(recursive_resolve(node->children_[0], btw_params[BTW_PARAM_NUM]))) {
SQL_RESV_LOG(WARN, "resolve child expr failed", K(ret), K(BTW_PARAM_NUM));
} else if (OB_ISNULL(btw_params[BTW_PARAM_NUM])) {
LOG_WARN("unexpected null", K(ret), K(BTW_PARAM_NUM));
}
}
}
if (OB_SUCC(ret)) {
if ((lib::is_oracle_mode() || can_transform_in_mysql_mode)) {
// convert between expr to >= and <=