Bug #32664 events.test fails randomly
Submitted: 23 Nov 2007 15:51 Modified: 25 Feb 2008 18:01
Reporter: Ingo Strüwing Email Updates:
Status: Closed Impact on me:
None 
Category:Tests: Server Severity:S7 (Test Cases)
Version:5.1.23 OS:Any
Assigned to: Horst Hunger CPU Architecture:Any
Tags: pbfail

[23 Nov 2007 15:51] Ingo Strüwing
Description:
main.events                    [ fail ]  timeout

Stopping All Servers

Disabling the test case. Please re-enable after fix.

How to repeat:
See pushbuild, for example https://intranet.mysql.com/secure/pushbuild/showdir.pl?dir=mysql-5.1-engines
log https://intranet.mysql.com/secure/pushbuild/getlog.pl?dir=mysql-5.1-engines&entry=rkalimul...
See also https://intranet.mysql.com/secure/pushbuild/xref.pl?startdate=&enddate=&dir=&plat=&testtyp...
[28 Nov 2007 10:23] Bugs System
Pushed into 6.0.4-alpha
[28 Nov 2007 10:25] Bugs System
Pushed into 5.1.23-rc
[20 Feb 2008 13:37] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/42649

ChangeSet@1.2531, 2008-02-20 14:40:46+01:00, hhunger@hh-nb.hungers +8 -0
  Patch for bug#32664: event.test has timeout randomly. File is split into event_1.test and event_2.test.
[22 Feb 2008 12:50] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/42823

ChangeSet@1.2532, 2008-02-22 13:54:49+01:00, hhunger@hh-nb.hungers +2 -0
  Replaced all --sleep by wait_condition (bug#32664).
[22 Feb 2008 18:55] Matthias Leich
Hi Horst,
I reviewed both Changesets.
1. I made stability checks with several test runs and
   high CPU load and/or high I/O load and both event
   test scripts never failed.
2. Code inspection:
   - I have no objections against the split of the old script.
   - events_1.test:
     Please insert a wait_condition before the SELECT in line 127
     because I fear this can fail in very rare cases.
     I have problems with the new comment in line 311, wrong
     grammar and what do you mean? I propose to remove the comment.
   - events_2.test
     Please replace the error numbers within the lines 199 and 201
     with error number names.
Ok to push after the proposed modifications above.
[22 Feb 2008 20:25] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/42874

ChangeSet@1.2537, 2008-02-22 21:28:59+01:00, hhunger@hh-nb.hungers +2 -0
  Inserted changes from the review of bug#32664.
[22 Feb 2008 20:56] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/42879

ChangeSet@1.2566, 2008-02-22 22:00:16+01:00, hhunger@hh-nb.hungers +1 -0
  Post merge to bug#32664.
[25 Feb 2008 15:58] Bugs System
Pushed into 5.1.24-rc
[25 Feb 2008 16:04] Bugs System
Pushed into 6.0.5-alpha
[25 Feb 2008 18:01] Paul DuBois
Test case changes. No changelog entry needed.