Merge lp:~percona-toolkit-dev/percona-toolkit/fix-1079341-pt-osc-skip-myisam-fk-checks into lp:percona-toolkit/2.1

Proposed by Brian Fraser
Status: Merged
Approved by: Daniel Nichter
Approved revision: 461
Merged at revision: 472
Proposed branch: lp:~percona-toolkit-dev/percona-toolkit/fix-1079341-pt-osc-skip-myisam-fk-checks
Merge into: lp:percona-toolkit/2.1
Diff against target: 14 lines (+5/-0)
1 file modified
bin/pt-online-schema-change (+5/-0)
To merge this branch: bzr merge lp:~percona-toolkit-dev/percona-toolkit/fix-1079341-pt-osc-skip-myisam-fk-checks
Reviewer Review Type Date Requested Status
Daniel Nichter Approve
Review via email: mp+134671@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
1=== modified file 'bin/pt-online-schema-change'
2--- bin/pt-online-schema-change 2012-11-13 15:22:01 +0000
3+++ bin/pt-online-schema-change 2012-11-16 14:31:29 +0000
4@@ -9129,6 +9129,11 @@
5 die "I need a $arg argument" unless $args{$arg};
6 }
7 my ($tbl, $cxn, $q) = @args{@required_args};
8+
9+ if ( lc($tbl->{tbl_struct}->{engine} || '') eq 'myisam' ) {
10+ PTDEBUG && _d(q{MyISAM table, not looking for child tables});
11+ return;
12+ }
13
14 PTDEBUG && _d('Finding child tables');
15

Subscribers

People subscribed via source and target branches