17 Jul
2023
17 Jul
'23
1:59 p.m.
Hi!
Sergei Golubchik <serg@mariadb.org> schrieb am Mo., 17. Juli 2023, 13:11:
> Hi, Sanja,
>
> The patch is ok, but the test case is rather weird.
> What does it do?
Magic (I don't digged deep, because it is very fast).
> How long does it run?
>
As I told very fast, less then a second.
>
> On Jul 17, Oleksandr Byelkin wrote:
> > revision-id: 845fbe9ce01 (mariadb-10.4.30-30-g845fbe9ce01)
> > parent(s): 02cd3675c4d
> > author: Oleksandr Byelkin
> > committer: Oleksandr Byelkin
> > timestamp: 2023-07-17 10:22:54 +0200
> > message:
> >
> > MDEV-30820 slow log Rows_examined out of range
> >
> > Fix row counters to be able to get any possible value.
> >
> > diff --git a/mysql-test/main/log_slow.test
> b/mysql-test/main/log_slow.test
> > --- a/mysql-test/main/log_slow.test
> > +++ b/mysql-test/main/log_slow.test
> > @@ -119,3 +119,98 @@ drop table t;
> > --echo #
> > --echo # End of 10.3 tests
> > --echo #
> > +
> > +
> > +--echo #
> > +--echo # MDEV-30820: slow log Rows_examined out of range
> > +--echo #
> > +
> > +set @log_output.save= @@global.log_output, @slow_log.save=
> @@global.slow_query_log;
> > +set global log_output= 'TABLE', slow_query_log= ON;
> > +set long_query_time= 0.000001;
> > +create database db;
> > +use db;
> > +
> > +--delimiter //
> > +CREATE OR REPLACE FUNCTION `get_id`(`INPUT_STRING` VARCHAR(50),
> `DECRYPT_KEY` CHAR(20)) RETURNS int(11)
> > +BEGIN
> > + DECLARE
> > + REQUEST_ID INT ;
> > + SET
> > + REQUEST_ID =
> > + IF(
> > + (
> > + LOCATE('#$#$#$ LHP_ID_', INPUT_STRING)
> > + ) > 0 AND
> > + (
> > + INPUT_STRING REGEXP DECRYPT_KEY
> > + )>0,
> > + (
> > + SPLIT_STRING(
> > + SUBSTRING_INDEX(
> > + SUBSTRING_INDEX(INPUT_STRING, '#$#$#$ LHP_ID_',
> -1),
> > + '#$#$#$',
> > + 1
> > + ),
> > + '-',
> > + 2
> > + )
> > + ),
> > + NULL
> > + ) ;
> > + SET
> > + REQUEST_ID=IF(
> > + ISNULL(REQUEST_ID)OR(REQUEST_ID='')OR(REQUEST_ID=0),
> > + NULL,
> > + REQUEST_ID
> > +
> > + );
> > + RETURN(REQUEST_ID) ;
> > +END
> > +//
> > +
> > +CREATE OR REPLACE FUNCTION `SPLIT_STRING`(`str` VARCHAR(255), `delim`
> VARCHAR(12), `pos` INT) RETURNS varchar(255) CHARSET utf8mb4
> > +RETURN REPLACE(
> > + SUBSTRING(
> > + SUBSTRING_INDEX(str , delim , pos) ,
> > + CHAR_LENGTH(
> > + SUBSTRING_INDEX(str , delim , pos - 1)
> > + ) + 1
> > + ) ,
> > + delim ,
> > + ''
> > +)
> > +//
> > +
> > +CREATE TABLE `tab1` (
> > +`ID` int(11) NOT NULL AUTO_INCREMENT,
> > +`NAME_F` varchar(50) DEFAULT NULL,
> > + PRIMARY KEY (`ID`)
> > +) DEFAULT CHARSET=utf8mb4 //
> > +
> > + CREATE TABLE `tab2` (
> > + `ID` int(11) NOT NULL AUTO_INCREMENT,
> > + `TAB1_ID` int(11) DEFAULT NULL,
> > + PRIMARY KEY (`id`)
> > +) DEFAULT CHARSET=utf8mb4 //
> > +
> > +for i in 1..100 do insert into tab1 values (null,'qwerty'); end for ; //
> > +for i in 1..1000 do insert into tab2 values (null,round(rand()*10000));
> end for ; //
> > +
> > +--delimiter ;
> > +
> > +SELECT
> > + get_id(CAST(aes_decrypt(tab1.NAME_F,'V41iNM0n4') AS
> char),'KM_ID_PL') as get_string,
> > + (CASE WHEN (SELECT ID FROM tab2 where tab2.TAB1_ID = tab1.ID LIMIT
> 1) IS NULL THEN 0 ELSE 1 END) AS IS_ATT
> > +FROM
> > + tab1
> > +ORDER BY 2 DESC
> > +LIMIT 2;
> > +
> > +set global log_output= @log_output.save, slow_query_log= @slow_log.save;
> > +drop database db;
> > +
> > +--echo #
> > +--echo # End of 10.4 tests
> > +--echo #
>
> Regards,
> Sergei
> VP of MariaDB Server Engineering
> and security@mariadb.org
>