Merge lp://staging/~stewart/percona-server/slave-timeout-fix-5.5 into lp://staging/percona-server/5.5

Proposed by Stewart Smith
Status: Merged
Approved by: Stewart Smith
Approved revision: no longer in the source branch.
Merged at revision: 128
Proposed branch: lp://staging/~stewart/percona-server/slave-timeout-fix-5.5
Merge into: lp://staging/percona-server/5.5
Diff against target: 24 lines (+12/-0)
2 files modified
series (+1/-0)
slave_timeout_fix.patch (+11/-0)
To merge this branch: bzr merge lp://staging/~stewart/percona-server/slave-timeout-fix-5.5
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+65350@code.staging.launchpad.net

Description of the change

This looks to be a correct fix... the timeout was certainly wrong, and we could have timing/scheduling fun on EC2 hosts that I guess could let things pause for a while (especially because of mtr --parallel). I've run some jenkins jobs with this patch and I seem to be having more success....

http://jenkins.percona.com/view/Percona%20Server%205.5/job/percona-server-5.5-param/11/

To post a comment you must log in.
Revision history for this message
Stewart Smith (stewart) wrote :

Also running http://jenkins.percona.com/view/Percona%20Server%205.5/job/percona-server-5.5-param/12/ which is the same branch again to see if we hit one of these failures again.

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches