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

Fix range calculation of ObDecimalInt

Browse Source
This commit is contained in:
obdev
2024-02-09 06:34:53 +00:00
committed by ob-robot
parent 89398dc586
commit 604130543f
1 changed files with 25 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
src/sql/rewrite/ob_query_range.cpp
View File

@ -23,6 +23,7 @@
#include "sql/optimizer/ob_optimizer_util.h"
#include "observer/omt/ob_tenant_srs.h"
#include "sql/engine/expr/ob_geo_expr_utils.h"
#include "sql/engine//expr/ob_datum_cast.h"
//if cnd is true get full range key part which is always true
//else, get empty key part which is always false
@ -6146,6 +6147,14 @@ OB_NOINLINE int ObQueryRange::cold_cast_cur_node(const ObKeyPart *cur,
ObAccuracy res_acc;
if (cur->pos_.column_type_.is_decimal_int()) {
res_acc = cur->pos_.column_type_.get_accuracy();
ObScale in_scale = cur_val.get_scale();
int32_t in_bytes = cur_val.get_int_bytes();
ObScale out_scale = res_acc.get_scale();
int32_t out_bytes = wide::ObDecimalIntConstValue::get_int_bytes_by_precision(res_acc.get_precision());
if (ObDatumCast::need_scale_decimalint(in_scale, in_bytes, out_scale, out_bytes)) {
// simply get range, using eq const mode
cast_ctx.cast_mode_ |= CM_CONST_TO_DECIMAL_INT_EQ;
}
cast_ctx.res_accuracy_ = &res_acc;
}
EXPR_CAST_OBJ_V2(expect_type, cur_val, dest_val);
@ -6890,9 +6899,14 @@ if (OB_SUCC(ret) ) { \
if (!start.is_min_value() && !start.is_max_value() && !start.is_unknown() \
&& (!ObSQLUtils::is_same_type_for_compare(start.get_meta(), column_type.get_obj_meta()) || start.is_decimal_int())) { \
ObCastMode cm = CM_WARN_ON_FAIL;\
if (ObDecimalIntType == expect_type.get_type()\
&& start.get_int_bytes() > wide::ObDecimalIntConstValue::get_int_bytes_by_precision(column_type.get_accuracy().get_precision())) {\
cm |= CM_CONST_TO_DECIMAL_INT_EQ;\
if (ObDecimalIntType == expect_type.get_type() && start.is_decimal_int()) {\
int32_t in_bytes = wide::ObDecimalIntConstValue::get_int_bytes_by_precision(column_type.get_accuracy().get_precision());\
ObScale in_scale = column_type.get_accuracy().get_scale();\
int32_t out_bytes = start.get_int_bytes();\
ObScale out_scale = start.get_scale();\
if (ObDatumCast::need_scale_decimalint(in_scale, in_bytes, out_scale, out_bytes)) {\
cm |= ObRelationalExprOperator::get_const_cast_mode(T_OP_GE, true);\
}\
}\
ObCastCtx cast_ctx(&allocator, &dtc_params, cm, expect_type.get_collation_type()); \
if (ObDecimalIntType == expect_type.get_type()) {\
@ -6935,9 +6949,14 @@ if (OB_SUCC(ret) ) { \
if (!end.is_min_value() && !end.is_max_value() && !end.is_unknown() \
&& (!ObSQLUtils::is_same_type_for_compare(end.get_meta(), column_type.get_obj_meta()) || end.is_decimal_int())) { \
ObCastMode cm = CM_WARN_ON_FAIL;\
if (ObDecimalIntType == expect_type.get_type()\
&& end.get_int_bytes() > wide::ObDecimalIntConstValue::get_int_bytes_by_precision(column_type.get_accuracy().get_precision())) {\
cm |= CM_CONST_TO_DECIMAL_INT_EQ;\
if (ObDecimalIntType == expect_type.get_type() && end.is_decimal_int()) {\
int32_t in_bytes = wide::ObDecimalIntConstValue::get_int_bytes_by_precision(column_type.get_accuracy().get_precision());\
ObScale in_scale = column_type.get_accuracy().get_scale();\
int32_t out_bytes = start.get_int_bytes();\
ObScale out_scale = start.get_scale();\
if (ObDatumCast::need_scale_decimalint(in_scale, in_bytes, out_scale, out_bytes)) {\
cm |= ObRelationalExprOperator::get_const_cast_mode(T_OP_LE, true);\
}\
}\
ObCastCtx cast_ctx(&allocator, &dtc_params, cm, expect_type.get_collation_type()); \
if (ObDecimalIntType == expect_type.get_type()) {\