mirror of
https://github.com/FirebirdSQL/firebird-qa.git
synced 2025-01-22 13:33:07 +01:00
Added/Updated tests\functional\tabloid\test_aae2ae32.py: Checked on 5.0.2.1516-92316F0 -- all ok: two hash joins instead of subquery.
This commit is contained in:
pavel-zotov
parent
b9c0379c0d
commit
44dd2cdb34
143
tests/functional/tabloid/test_aae2ae32.py
Normal file
143
tests/functional/tabloid/test_aae2ae32.py
Normal file
@ -0,0 +1,143 @@
|
||||
#coding:utf-8
|
||||
|
||||
"""
|
||||
ID: issue-8252
|
||||
ISSUE: https://github.com/FirebirdSQL/firebird/commit/aae2ae3291855f51ff587d0da055aed270137e8f
|
||||
TITLE: Better fix for #8252: incorrect subquery unnesting with complex dependencies, it re-allows converting nested IN/EXISTS into multiple semi-join
|
||||
DESCRIPTION:
|
||||
Test verifies additional commit related to https://github.com/FirebirdSQL/firebird/issues/8252
|
||||
Example #1 from https://github.com/FirebirdSQL/firebird/issues/8265 is used for check
|
||||
(suggested by dimitr, letter 25.09.2024 13:33)
|
||||
NOTES:
|
||||
[26.09.2024] pzotov
|
||||
1. No ticket has been created for this test.
|
||||
2. Confirmed problem on 5.0.2.1516-fe6ba50 (23.09.2024), got for SubQueryConversion = true plan with subquery:
|
||||
# Sub-query
|
||||
# -> Filter
|
||||
# -> Filter
|
||||
# -> Hash Join (semi)
|
||||
# ... <TEST2> and <TEST3> ...
|
||||
# Select Expression
|
||||
# -> Aggregate
|
||||
# -> Filter
|
||||
# -> Table "TEST1" as "A" Full Scan
|
||||
3. Parameter 'SubQueryConversion' currently presents only in FB 5.x and _NOT_ in FB 6.x.
|
||||
Because of that, testing version are limited only for 5.0.2. FB 6.x currently is NOT tested.
|
||||
4. Custom driver config objects are created here, one with SubQueryConversion = true and second with false.
|
||||
|
||||
Checked on 5.0.2.1516-92316F0 -- all ok: two hash joins instead of subquery.
|
||||
Thanks to dimitr for the advice on implementing the test.
|
||||
"""
|
||||
|
||||
import pytest
|
||||
from firebird.qa import *
|
||||
from firebird.driver import driver_config, connect, DatabaseError
|
||||
|
||||
init_script = """
|
||||
create table test1(id int not null);
|
||||
create table test2(id int not null, pid int not null);
|
||||
create table test3(id int not null, pid int not null, name varchar(30) not null);
|
||||
commit;
|
||||
|
||||
insert into test1(id) select row_number()over()-1 from rdb$types rows 10;
|
||||
insert into test2(id, pid) select row_number()over()-1, mod(row_number()over()-1, 10) from rdb$types rows 100;
|
||||
insert into test3(id, pid, name) select row_number()over()-1, mod(row_number()over()-1, 100), 'QWEABCRTY' from rdb$types, rdb$types rows 1000;
|
||||
commit;
|
||||
|
||||
-- alter table test1 add constraint test1_pk primary key(id);
|
||||
-- alter table test2 add constraint test2_pk primary key(id);
|
||||
-- alter table test3 add constraint test3_pk primary key(id);
|
||||
-- alter table test2 add constraint test2_fk foreign key(pid) references test1;
|
||||
-- alter table test3 add constraint test3_fk foreign key(pid) references test2;
|
||||
"""
|
||||
|
||||
db = db_factory(init=init_script)
|
||||
|
||||
act = python_act('db', substitutions = [('record length: \\d+', 'record length')])
|
||||
|
||||
#-----------------------------------------------------------
|
||||
|
||||
def replace_leading(source, char="."):
|
||||
stripped = source.lstrip()
|
||||
return char * (len(source) - len(stripped)) + stripped
|
||||
|
||||
#-----------------------------------------------------------
|
||||
|
||||
@pytest.mark.version('>=5.0.2,<6')
|
||||
def test_1(act: Action, capsys):
|
||||
|
||||
test_sql = """
|
||||
select count(*) from test1 a
|
||||
where
|
||||
a.id in (
|
||||
select b.pid from test2 b
|
||||
where
|
||||
b.id in (
|
||||
select c.pid from test3 c
|
||||
where name like '%ABC%'
|
||||
)
|
||||
);
|
||||
"""
|
||||
|
||||
for sq_conv in ('true','false',):
|
||||
srv_cfg = driver_config.register_server(name = f'srv_cfg_aae2ae32_{sq_conv}', config = '')
|
||||
db_cfg_name = f'db_cfg_aae2ae32_{sq_conv}'
|
||||
db_cfg_object = driver_config.register_database(name = db_cfg_name)
|
||||
db_cfg_object.server.value = srv_cfg.name
|
||||
db_cfg_object.database.value = str(act.db.db_path)
|
||||
db_cfg_object.config.value = f"""
|
||||
SubQueryConversion = {sq_conv}
|
||||
"""
|
||||
|
||||
with connect(db_cfg_name, user = act.db.user, password = act.db.password) as con:
|
||||
try:
|
||||
cur = con.cursor()
|
||||
cur.execute("select g.rdb$config_name, g.rdb$config_value from rdb$database r left join rdb$config g on g.rdb$config_name = 'SubQueryConversion'")
|
||||
for r in cur:
|
||||
print(r[0],r[1])
|
||||
|
||||
ps = cur.prepare(test_sql)
|
||||
|
||||
# Print explained plan with padding eash line by dots in order to see indentations:
|
||||
print( '\n'.join([replace_leading(s) for s in ps.detailed_plan.split('\n')]) )
|
||||
|
||||
# Print data:
|
||||
for r in cur.execute(ps):
|
||||
print(r[0])
|
||||
except DatabaseError as e:
|
||||
print(e.__str__())
|
||||
print(e.gds_codes)
|
||||
|
||||
act.expected_stdout = f"""
|
||||
SubQueryConversion true
|
||||
Select Expression
|
||||
....-> Aggregate
|
||||
........-> Filter
|
||||
............-> Hash Join (semi)
|
||||
................-> Table "TEST1" as "A" Full Scan
|
||||
................-> Record Buffer (record length: 82111)
|
||||
....................-> Filter
|
||||
........................-> Hash Join (semi)
|
||||
............................-> Table "TEST2" as "B" Full Scan
|
||||
............................-> Record Buffer (record length: 57111)
|
||||
................................-> Filter
|
||||
....................................-> Table "TEST3" as "C" Full Scan
|
||||
10
|
||||
|
||||
SubQueryConversion false
|
||||
Sub-query
|
||||
....-> Filter
|
||||
........-> Filter
|
||||
............-> Table "TEST3" as "C" Full Scan
|
||||
Sub-query
|
||||
....-> Filter
|
||||
........-> Filter
|
||||
............-> Table "TEST2" as "B" Full Scan
|
||||
Select Expression
|
||||
....-> Aggregate
|
||||
........-> Filter
|
||||
............-> Table "TEST1" as "A" Full Scan
|
||||
10
|
||||
"""
|
||||
act.stdout = capsys.readouterr().out
|
||||
assert act.clean_stdout == act.clean_expected_stdout
|
||||
Loading…
Reference in New Issue
Block a user