diff options
author | Alexander Barkov <bar@mariadb.com> | 2020-07-29 10:22:17 +0400 |
---|---|---|
committer | Alexander Barkov <bar@mariadb.com> | 2020-07-29 21:45:41 +0400 |
commit | 92499ae95ced000b064910d1a15705faa64cc88f (patch) | |
tree | cf260fa9fc647fb6ed52530c29a46006a73e78cb /sql/sql_binlog.h | |
parent | 423de1e57417281a8b7265b97f46bf3933e411ad (diff) | |
download | mariadb-git-92499ae95ced000b064910d1a15705faa64cc88f.tar.gz |
MDEV-23320 Hex hybrid constants 0xHHHH work badly in rounding functions
- Type_handler_hex_hybrid did not override
Type_handler_string_result::Item_func_round_fix_length_and_dec(),
so the result type of ROUND(0xFFFFFFFFFFFFFFFF) was erroneously
calculated ad DOUBLE with a wrong length.
Overriding Item_func_round_fix_length_and_dec(), to calculated
the result type as INT/BIGINT.
Also, fixing Item_func_round::fix_arg_int() to use
args[0]->decimal_precision() instead of args[0]->max_length
when calculating this->max_length, to get a correct result
for hex hybrids.
- Type_handler_hex_hybrid::Item_func_int_val_fix_length_and_dec()
called item->fix_length_and_dec_int_or_decimal(), which did not
produce a correct result data type for hex hybrid.
Implementing a dedicated code instead, to return INT UNSIGNED or
BIGINT UNSIGNED depending in the number of digits in the arguments.
Diffstat (limited to 'sql/sql_binlog.h')
0 files changed, 0 insertions, 0 deletions