Commit d09ed4e0 authored by Matthias Leich's avatar Matthias Leich

Fix for Bug#47745 innodb.innodb-timeout fails sporadically

- Second scenario checked:
  Ensure via wait routines that the commit comes after the
  processing of the statement which should get finally
  the ER_LOCK_WAIT_TIMEOUT
  --> This should prevent the current bug.
- First scenario checked:
  Ensure via wait routines that the statement is already waiting
  for getting the lock before the commit is given.
  --> No effect on the current bug, but ensure that the right
      scenario is reached.
- Take care that disconnects are finished before the test ends.
  --> Reduce the potential to harm succeeding tests.
- "Mangle" the printout of the current default innodb_lock_wait_timeout value
  --> No need to adjust the test in case the default gets changed in future.
parent a79e8fa2
...@@ -13,13 +13,14 @@ set global innodb_lock_wait_timeout=347; ...@@ -13,13 +13,14 @@ set global innodb_lock_wait_timeout=347;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
@@innodb_lock_wait_timeout @@innodb_lock_wait_timeout
42 42
set innodb_lock_wait_timeout=1; set innodb_lock_wait_timeout=10;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
@@innodb_lock_wait_timeout @@innodb_lock_wait_timeout
1 10
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
@@innodb_lock_wait_timeout @@innodb_lock_wait_timeout
347 347
SET @connection_b_id = <connection_b_id>;
create table t1(a int primary key)engine=innodb; create table t1(a int primary key)engine=innodb;
begin; begin;
insert into t1 values(1),(2),(3); insert into t1 values(1),(2),(3);
...@@ -31,8 +32,9 @@ a ...@@ -31,8 +32,9 @@ a
3 3
begin; begin;
insert into t1 values(4); insert into t1 values(4);
set innodb_lock_wait_timeout=3;
select * from t1 for update; select * from t1 for update;
commit; commit;
ERROR HY000: Lock wait timeout exceeded; try restarting transaction ERROR HY000: Lock wait timeout exceeded; try restarting transaction
drop table t1; drop table t1;
set global innodb_lock_wait_timeout=50; set global innodb_lock_wait_timeout=<initial_timeout>;
-- source include/have_innodb.inc -- source include/have_innodb.inc
let $timeout=`select @@innodb_lock_wait_timeout`; let $initial_timeout=`select @@innodb_lock_wait_timeout`;
set global innodb_lock_wait_timeout=42; set global innodb_lock_wait_timeout=42;
connect (a,localhost,root,,); connect (a,localhost,root,,);
...@@ -12,19 +12,24 @@ set innodb_lock_wait_timeout=1; ...@@ -12,19 +12,24 @@ set innodb_lock_wait_timeout=1;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
connection b; connection b;
let $connection_b_id=`SELECT CONNECTION_ID()`;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
set global innodb_lock_wait_timeout=347; set global innodb_lock_wait_timeout=347;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
set innodb_lock_wait_timeout=1; set innodb_lock_wait_timeout=10;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
connect (c,localhost,root,,); connect (c,localhost,root,,);
connection c; connection c;
select @@innodb_lock_wait_timeout; select @@innodb_lock_wait_timeout;
connection default;
disconnect c; disconnect c;
--source include/wait_until_disconnected.inc
connection a; connection a;
--replace_result $connection_b_id <connection_b_id>
eval SET @connection_b_id = $connection_b_id;
create table t1(a int primary key)engine=innodb; create table t1(a int primary key)engine=innodb;
begin; begin;
insert into t1 values(1),(2),(3); insert into t1 values(1),(2),(3);
...@@ -33,7 +38,37 @@ connection b; ...@@ -33,7 +38,37 @@ connection b;
--send --send
select * from t1 for update; select * from t1 for update;
# Observation on information_schema.processlist (2010-12 mysql-5.5)
# -----------------------------------------------------------------
# As soon as the server started the execution of the
# connection a: --send select ... for update
# High parallel load could delay this up to two seconds.
# and before either
# - the innodb_lock_wait_timeout was exceeded
# -> connection b reap gets ER_LOCK_WAIT_TIMEOUT
# or
# - connection a commits, the lock disappears and the statement
# of connection b finishes
# -> connection b reap gets success + result set
# we see within information_schema.processlist for connection b a row
# command state info
# Query Sending data select * from t1 for update
# The highest time value seen was @@innodb_lock_wait_timeout + 1.
# Please note that there is unfortunately nothing which says
# that we are just waiting for a lock.
connection a; connection a;
# In order to ensure that the execution of
# connection b: select * from t1 for update
# has really started and is most probably waiting for the lock now we poll on
# information_schema.processlist.
# Also our current session innodb_lock_wait_timeout of 10 seconds should big
# enough to prevent that connection b ends up with getting ER_LOCK_WAIT_TIMEOUT.
#
let $wait_timeout= 10;
let $wait_condition= SELECT COUNT(*) = 1 FROM information_schema.processlist
WHERE id = @connection_b_id AND INFO = 'select * from t1 for update';
--source include/wait_condition.inc
commit; commit;
connection b; connection b;
...@@ -44,21 +79,39 @@ begin; ...@@ -44,21 +79,39 @@ begin;
insert into t1 values(4); insert into t1 values(4);
connection b; connection b;
set innodb_lock_wait_timeout=3;
# 3 seconds should be big enough that the wait routine of connection a will
# hit the time span where our next statement is visible within the
# information_schema.processlist.
--send --send
select * from t1 for update; select * from t1 for update;
connection a; connection a;
sleep 2; # Wait till the execution of the connection b statement was started.
let $wait_timeout= 10;
let $wait_condition= SELECT COUNT(*) = 1 FROM information_schema.processlist
WHERE id = @connection_b_id AND INFO = 'select * from t1 for update';
--source include/wait_condition.inc
# Wait till the execution of the connection b statement has ended.
let $wait_timeout= 10;
let $wait_condition= SELECT COUNT(*) = 1 FROM information_schema.processlist
WHERE id = @connection_b_id AND INFO IS NULL;
--source include/wait_condition.inc
# Give "commit" though this must be too late for the statement of connection b.
commit; commit;
connection b; connection b;
--error ER_LOCK_WAIT_TIMEOUT --error ER_LOCK_WAIT_TIMEOUT
reap; reap;
drop table t1;
connection default; disconnect b;
--source include/wait_until_disconnected.inc
connection a;
disconnect a; disconnect a;
disconnect b; --source include/wait_until_disconnected.inc
eval set global innodb_lock_wait_timeout=$timeout; connection default;
drop table t1;
--replace_result $initial_timeout <initial_timeout>
eval set global innodb_lock_wait_timeout=$initial_timeout;
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment