[MDEV-30542] Update the spider self_reference test to include a case of circular referencing of length > 0 - Jira

Details

Type: Task
Status: Closed (View Workflow)
Priority: Major
Resolution: Fixed
Fix Version/s: 11.1.1, 10.11.3, 11.0.2, 10.5.20, 10.6.13, 10.8.8, 10.9.6, 10.10.4
Component/s: Storage Engine - Spider
Labels:
None

Description

The self_reference test added in commit 23c8adda749 for ~~MDEV-6268~~ only contains a case of a table directly referencing itself (length 0) which is not very realistic. The loop detection feature added there is believed to be more powerful than that, and can detect arbitrary lengths. Update the testcase to cover this case.

Attachments

Issue Links

relates to

MDEV-6268 SPIDER table with no COMMENT clause causes queries to wait forever

Closed

Activity

Ascending order - Click to sort in descending order

Yuchen Pei created issue - 2023-02-02 01:39

Yuchen Pei made changes - 2023-02-02 01:39

Field	Original Value	New Value
Link		This issue relates to ~~MDEV-6268~~ [ ~~MDEV-6268~~ ]

Yuchen Pei made changes - 2023-02-02 01:40

Description

The self_reference test added in commit 23c8adda749 for ~~MDEV-6268~~ only contains a case of a table connecting to itself (length 0) which is not very realistic. The loop detection feature added there is believed to be more powerful than that, and can detect arbitrary lengths. Update the testcase to cover this case.

Yuchen Pei made changes - 2023-02-02 01:41

Fix Version/s		10.5 [ 23123 ]
Fix Version/s		10.6 [ 24028 ]
Fix Version/s		10.7 [ 24805 ]
Fix Version/s		10.8 [ 26121 ]
Fix Version/s		10.9 [ 26905 ]
Fix Version/s		10.10 [ 27530 ]
Fix Version/s		10.11 [ 27614 ]
Fix Version/s		11.0 [ 28320 ]

Yuchen Pei made changes - 2023-02-02 01:41

Status

Open [ 1 ]

In Progress [ 3 ]

Yuchen Pei added a comment - 2023-02-02 02:18 - edited

I thought it would be somewhat trivial to write a test case for this, at least when all tables are on the same server, but I'm getting mysqltest: At line 27: query 'SELECT a FROM tbl_a' failed: <Unknown> (12720): Host:localhost and Socket:/home/ycp/source/mariadb-server/mdev-30542/build/mysql-test/var/tmp/mysqld.1.1.sock aim self server. Please change spider_same_server_link parameter if this link is required. despite spider_same_server_link is on for the following case.

--disable_query_log

--disable_result_log

--source ../../t/test_init.inc

--enable_result_log

--enable_query_log

--connection master_1

CREATE DATABASE auto_test_local;

USE auto_test_local;

SET spider_same_server_link= on;

eval CREATE TABLE tbl_a (

    a INT

) $MASTER_1_ENGINE COMMENT='table "tbl_b", srv "s_1"';

eval CREATE TABLE tbl_b (

    a INT

) $MASTER_1_ENGINE COMMENT='table "tbl_c", srv "s_1"';

eval CREATE TABLE tbl_c (

    a INT

) $MASTER_1_ENGINE COMMENT='table "tbl_a", srv "s_1"';

--error 12719

SELECT a FROM tbl_a;

--connection master_1

DROP DATABASE IF EXISTS auto_test_local;

--disable_query_log

--disable_result_log

--source ../t/test_deinit.inc

--enable_query_log

--enable_result_log

Yuchen Pei added a comment - 2023-02-02 02:18 - edited I thought it would be somewhat trivial to write a test case for this, at least when all tables are on the same server, but I'm getting mysqltest: At line 27: query 'SELECT a FROM tbl_a' failed: <Unknown> (12720): Host:localhost and Socket:/home/ycp/source/mariadb-server/mdev-30542/build/mysql-test/var/tmp/mysqld.1.1.sock aim self server. Please change spider_same_server_link parameter if this link is required. despite spider_same_server_link is on for the following case. --disable_query_log --disable_result_log --source ../../t/test_init.inc --enable_result_log --enable_query_log --connection master_1 CREATE DATABASE auto_test_local; USE auto_test_local; SET spider_same_server_link= on ; eval CREATE TABLE tbl_a ( a INT ) $MASTER_1_ENGINE COMMENT= 'table "tbl_b", srv "s_1"' ; eval CREATE TABLE tbl_b ( a INT ) $MASTER_1_ENGINE COMMENT= 'table "tbl_c", srv "s_1"' ; eval CREATE TABLE tbl_c ( a INT ) $MASTER_1_ENGINE COMMENT= 'table "tbl_a", srv "s_1"' ; --error 12719 SELECT a FROM tbl_a; --connection master_1 DROP DATABASE IF EXISTS auto_test_local; --disable_query_log --disable_result_log --source ../t/test_deinit.inc --enable_query_log --enable_result_log

Yuchen Pei made changes - 2023-02-02 02:18

Status

In Progress [ 3 ]

Stalled [ 10000 ]

Julien Fritsch made changes - 2023-03-03 17:33

Fix Version/s

10.7 [ 24805 ]

Yuchen Pei added a comment - 2023-04-03 23:59 - edited

Here's a simple test that works - place it under storage/spider/mysql-test/spider/bugfix/t/. I came up with it while working on ~~MDEV-29676~~.

--disable_query_log

--disable_result_log

--source ../../t/test_init.inc

--enable_result_log

--enable_query_log

eval CREATE SERVER srv FOREIGN DATA WRAPPER MYSQL OPTIONS (SOCKET "$MASTER_1_MYSOCK", DATABASE 'test',user 'root');

create table t2 (c int);

create table t1 (c int) ENGINE=Spider COMMENT='WRAPPER "mysql", srv "srv",TABLE "t2"';

create table t0 (c int) ENGINE=Spider COMMENT='WRAPPER "mysql", srv "srv",TABLE "t1"';

alter table t2 ENGINE=Spider COMMENT='WRAPPER "mysql", srv "srv",TABLE "t0"';

--error 12719

select * from t0;

--error 12719

select * from t1;

--error 12719

select * from t2;

drop table t0, t1, t2;

--disable_query_log

--disable_result_log

--source ../../t/test_deinit.inc

--enable_result_log

--enable_query_log

Yuchen Pei added a comment - 2023-04-03 23:59 - edited Here's a simple test that works - place it under storage/spider/mysql-test/spider/bugfix/t/ . I came up with it while working on MDEV-29676 . --disable_query_log --disable_result_log --source ../../t/test_init.inc --enable_result_log --enable_query_log eval CREATE SERVER srv FOREIGN DATA WRAPPER MYSQL OPTIONS (SOCKET "$MASTER_1_MYSOCK" , DATABASE 'test' , user 'root' ); create table t2 (c int ); create table t1 (c int ) ENGINE=Spider COMMENT= 'WRAPPER "mysql", srv "srv",TABLE "t2"' ; create table t0 (c int ) ENGINE=Spider COMMENT= 'WRAPPER "mysql", srv "srv",TABLE "t1"' ; alter table t2 ENGINE=Spider COMMENT= 'WRAPPER "mysql", srv "srv",TABLE "t0"' ; --error 12719 select * from t0; --error 12719 select * from t1; --error 12719 select * from t2; drop table t0, t1, t2; --disable_query_log --disable_result_log --source ../../t/test_deinit.inc --enable_result_log --enable_query_log

Yuchen Pei added a comment - 2023-04-14 01:09

Hi holyfoot, PTAL thanks

https://github.com/MariaDB/server/commit/fadb119fed8

Yuchen Pei added a comment - 2023-04-14 01:09 Hi holyfoot , PTAL thanks https://github.com/MariaDB/server/commit/fadb119fed8

Yuchen Pei made changes - 2023-04-14 01:09

Assignee	Yuchen Pei [ JIRAUSER52627 ]	Alexey Botchkov [ holyfoot ]
Status	Stalled [ 10000 ]	In Review [ 10002 ]

Alexey Botchkov added a comment - 2023-04-18 06:25

looks ok to push.

Alexey Botchkov added a comment - 2023-04-18 06:25 looks ok to push.

Alexey Botchkov made changes - 2023-04-18 06:25

Assignee	Alexey Botchkov [ holyfoot ]	Yuchen Pei [ JIRAUSER52627 ]
Status	In Review [ 10002 ]	Stalled [ 10000 ]

Yuchen Pei added a comment - 2023-04-19 01:55

Thanks for the review, pushed to 10.5

Yuchen Pei added a comment - 2023-04-19 01:55 Thanks for the review, pushed to 10.5

Yuchen Pei made changes - 2023-04-19 01:55

Fix Version/s		10.5.20 [ 28512 ]
Fix Version/s		10.6.13 [ 28514 ]
Fix Version/s		10.9.6 [ 28520 ]
Fix Version/s		10.10.4 [ 28522 ]
Fix Version/s		10.11.3 [ 28524 ]
Fix Version/s		11.1.1 [ 28704 ]
Fix Version/s		10.8.8 [ 28518 ]
Fix Version/s		11.0.2 [ 28706 ]
Fix Version/s	10.5 [ 23123 ]
Fix Version/s	10.6 [ 24028 ]
Fix Version/s	10.8 [ 26121 ]
Fix Version/s	10.9 [ 26905 ]
Fix Version/s	10.10 [ 27530 ]
Fix Version/s	10.11 [ 27614 ]
Fix Version/s	11.0 [ 28320 ]
Resolution		Fixed [ 1 ]
Status	Stalled [ 10000 ]	Closed [ 6 ]

Yuchen Pei added a comment - 2023-06-28 05:14

Pushed a trivial fixup https://github.com/MariaDB/server/commit/24faa5de16c

Yuchen Pei added a comment - 2023-06-28 05:14 Pushed a trivial fixup https://github.com/MariaDB/server/commit/24faa5de16c

People

Assignee:: Yuchen Pei

Reporter:: Yuchen Pei

Votes:: 0 Vote for this issue

Watchers:: 4 Start watching this issue

Dates

Created:: 2023-02-02 01:39

Updated:: 2023-07-26 04:31

Resolved:: 2023-04-19 01:55

Git Integration

Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.

MariaDB Server

Details

Description

Attachments

Issue Links

Activity

People

Dates

Git Integration