Skip to content

Commit

Permalink
Force ddl_high_priority.test to set count_sessions=1
Browse files Browse the repository at this point in the history
Summary:
The ddl_high_priority test still fails with occassional flakiness. Force
count_sessions to 1 and add an additional check to verify the number of
processes returned by processlist is 1.

Reviewed By: asandryh

Differential Revision: D6619947

fbshipit-source-id: 13815d1
  • Loading branch information
Herman Lee authored and facebook-github-bot committed Dec 21, 2017
1 parent a1ecd76 commit e9cb575
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion mysql-test/include/ddl_high_priority.inc
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ drop table t1_new;
## no effect for regular users
##

--source include/count_sessions.inc
let $count_sessions = 1;
connect (con2,localhost,test_user2,,test,,);
# $con_kill is regular user
--let $con_kill = con2
Expand All @@ -89,6 +89,10 @@ disconnect con2;

# Ensure con2 is closed
--source include/wait_until_count_sessions.inc
# Secondary check to ensure con2 is closed
let $wait_condition=
SELECT count(*) = 1 FROM information_schema.processlist;
--source include/wait_condition.inc

# restore $con_kill
--let $con_kill = default
Expand Down

0 comments on commit e9cb575

Please sign in to comment.