Merge lp:~percona-toolkit-dev/percona-toolkit/fix-dl-ts-bug-1195034-2.1 into lp:percona-toolkit/2.1

Proposed by Daniel Nichter
Status: Merged
Merged at revision: 533
Proposed branch: lp:~percona-toolkit-dev/percona-toolkit/fix-dl-ts-bug-1195034-2.1
Merge into: lp:percona-toolkit/2.1
Diff against target: 69 lines (+36/-0)
3 files modified
bin/pt-deadlock-logger (+5/-0)
t/pt-deadlock-logger/bugs.t (+17/-0)
t/pt-deadlock-logger/samples/bug_1195034.txt (+14/-0)
To merge this branch: bzr merge lp:~percona-toolkit-dev/percona-toolkit/fix-dl-ts-bug-1195034-2.1
Reviewer Review Type Date Requested Status
Daniel Nichter Approve
Review via email: mp+171682@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Daniel Nichter (daniel-nichter) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/pt-deadlock-logger'
--- bin/pt-deadlock-logger 2013-02-11 17:31:15 +0000
+++ bin/pt-deadlock-logger 2013-06-26 22:35:30 +0000
@@ -4140,6 +4140,11 @@
41404140
4141 # Extract some miscellaneous data from the deadlock.4141 # Extract some miscellaneous data from the deadlock.
4142 my ( $ts ) = $dl_text =~ m/^$s$/m;4142 my ( $ts ) = $dl_text =~ m/^$s$/m;
4143 if ( !$ts ) {
4144 # https://bugs.launchpad.net/percona-toolkit/+bug/1195034
4145 # 130624 17:39:24TOO DEEP OR LONG SEARCH IN THE LOCK TABLE ...
4146 ($ts) = $dl_text =~ m/^${s}TOO DEEP/m;
4147 }
4143 my ( $year, $mon, $day, $hour, $min, $sec ) = $ts =~ m/^((?:\d\d)?\d\d)-?(\d\d)-?(\d\d) +(\d+):(\d+):(\d+)$/;4148 my ( $year, $mon, $day, $hour, $min, $sec ) = $ts =~ m/^((?:\d\d)?\d\d)-?(\d\d)-?(\d\d) +(\d+):(\d+):(\d+)$/;
4144 if ( length($year) == 2 ) {4149 if ( length($year) == 2 ) {
4145 $year += 2000;4150 $year += 2000;
41464151
=== modified file 't/pt-deadlock-logger/bugs.t'
--- t/pt-deadlock-logger/bugs.t 2012-11-23 22:11:50 +0000
+++ t/pt-deadlock-logger/bugs.t 2013-06-26 22:35:30 +0000
@@ -15,6 +15,8 @@
15use Sandbox;15use Sandbox;
16require "$trunk/bin/pt-deadlock-logger";16require "$trunk/bin/pt-deadlock-logger";
1717
18use Data::Dumper;
19
18# #############################################################################20# #############################################################################
19# https://bugs.launchpad.net/percona-toolkit/+bug/90344321# https://bugs.launchpad.net/percona-toolkit/+bug/903443
20# pt-deadlock-logger crashes on MySQL 5.522# pt-deadlock-logger crashes on MySQL 5.5
@@ -120,6 +122,21 @@
120);122);
121123
122# #############################################################################124# #############################################################################
125# https://bugs.launchpad.net/percona-toolkit/+bug/1195034
126# pt-deadlock-logger error: Use of uninitialized value $ts in pattern match
127# #############################################################################
128
129$innodb_status_sample = load_file("t/pt-deadlock-logger/samples/bug_1195034.txt");
130my $deadlocks = pt_deadlock_logger::parse_deadlocks($innodb_status_sample);
131
132is_deeply(
133 $deadlocks,
134 {
135 },
136 "Bug 1195034: TOO DEEP OR LONG SEARCH IN THE LOCK TABLE WAITS-FOR GRAPH"
137) or diag(Dumper($deadlocks));
138
139# #############################################################################
123# Done.140# Done.
124# #############################################################################141# #############################################################################
125done_testing;142done_testing;
126143
=== added file 't/pt-deadlock-logger/samples/bug_1195034.txt'
--- t/pt-deadlock-logger/samples/bug_1195034.txt 1970-01-01 00:00:00 +0000
+++ t/pt-deadlock-logger/samples/bug_1195034.txt 2013-06-26 22:35:30 +0000
@@ -0,0 +1,14 @@
1
2------------------------
3LATEST DETECTED DEADLOCK
4------------------------
5130624 17:39:24TOO DEEP OR LONG SEARCH IN THE LOCK TABLE WAITS-FOR GRAPH, WE WILL ROLL BACK FOLLOWING TRANSACTION
6
7*** TRANSACTION:
8TRANSACTION 3BF88F886, ACTIVE 0 sec setting auto-inc lock
9mysql tables in use 1, locked 1
101 lock struct(s), heap size 376, 0 row lock(s)
11MySQL thread id 23512694, OS thread handle 0x5055b940, query id 734303798 10.10.10.1 host update
12INSERT INTO gr_v3_response_log (query_key, time_received, time_to_respond, status, raw_response, api_host, api_path, api_client) VALUES ('...
13*** WAITING FOR THIS LOCK TO BE GRANTED:
14TABLE LOCK table `db`.`gr_v3_response_log` trx id 3BF88F886 lock mode AUTO-INC waiting

Subscribers

People subscribed via source and target branches