Skip to content

Commit

Permalink
MDEV-32651: Lost Debug_sync signal in rpl_sql_thd_start_errno_cleared
Browse files Browse the repository at this point in the history
The test rpl.rpl_sql_thd_start_errno_cleared can lose a debug_sync
signal, as there is a RESET immediately following a SIGNAL. When the
signal is lost, the sql_thread is stuck in a WAIT_FOR clause until
it times out, resulting in long test times (albeit still
successful).

This patch extends the test to ensure the debug_sync signal was
received before issuing the RESET
  • Loading branch information
bnestere committed Nov 1, 2023
1 parent 4b65859 commit c341743
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ set debug_sync= "now wait_for sql_thread_run_lock_released";
# Validating that Last_SQL_Errno is cleared..
# ..success
set debug_sync= "now signal sql_thread_continue";
# Wait for debug_sync signal to have been received before issuing RESET
set @@global.debug_dbug= @saved_dbug;
set debug_sync= "RESET";
# Cleanup
Expand Down
4 changes: 4 additions & 0 deletions mysql-test/suite/rpl/t/rpl_sql_thd_start_errno_cleared.test
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,10 @@ if ($last_error)

set debug_sync= "now signal sql_thread_continue";

--echo # Wait for debug_sync signal to have been received before issuing RESET
let $wait_condition= select count(*)=0 from information_schema.processlist where state like "debug sync point%";
source include/wait_condition.inc;

set @@global.debug_dbug= @saved_dbug;
set debug_sync= "RESET";

Expand Down

0 comments on commit c341743

Please sign in to comment.