Merge lp:~percona-toolkit-dev/percona-toolkit/fix-pt-index-usage-db-bug-1028614 into lp:percona-toolkit/2.1

Proposed by Daniel Nichter
Status: Merged
Merged at revision: 328
Proposed branch: lp:~percona-toolkit-dev/percona-toolkit/fix-pt-index-usage-db-bug-1028614
Merge into: lp:percona-toolkit/2.1
Diff against target: 50 lines (+16/-4)
2 files modified
bin/pt-index-usage (+1/-1)
t/pt-index-usage/basics.t (+15/-3)
To merge this branch: bzr merge lp:~percona-toolkit-dev/percona-toolkit/fix-pt-index-usage-db-bug-1028614
Reviewer Review Type Date Requested Status
Daniel Nichter Approve
Review via email: mp+117169@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-index-usage'
2--- bin/pt-index-usage 2012-07-20 20:25:10 +0000
3+++ bin/pt-index-usage 2012-07-28 20:51:20 +0000
4@@ -5013,7 +5013,7 @@
5
6 my $schema = new Schema();
7 my $schema_itr = new SchemaIterator(
8- dbh => $dbh,
9+ dbh => $si_dbh,
10 OptionParser => $o,
11 Quoter => $q,
12 TableParser => $tp,
13
14=== modified file 't/pt-index-usage/basics.t'
15--- t/pt-index-usage/basics.t 2012-06-09 18:43:33 +0000
16+++ t/pt-index-usage/basics.t 2012-07-28 20:51:20 +0000
17@@ -27,9 +27,6 @@
18 if ( !@{ $dbh->selectall_arrayref("show databases like 'sakila'") } ) {
19 plan skip_all => "Sakila database is not loaded";
20 }
21-else {
22- plan tests => 9;
23-}
24
25 my $cnf = '/tmp/12345/my.sandbox.cnf';
26 my @args = ('-F', $cnf);
27@@ -136,8 +133,23 @@
28 'No output without default db'
29 );
30
31+# https://bugs.launchpad.net/percona-toolkit/+bug/1028614
32+$dbh->do("CREATE DATABASE IF NOT EXISTS z");
33+$dbh->do("CREATE TABLE z.t (id int)");
34+
35+ok(
36+ no_diff(
37+ sub { pt_index_usage::main(@args, qw(-D sakila),
38+ "$trunk/$samples/slow006.txt") },
39+ "$samples/slow006-report.txt"
40+ ),
41+ '--database is kept (bug 1028614)'
42+);
43+
44 # #############################################################################
45 # Done.
46 # #############################################################################
47+$sb->wipe_clean($dbh);
48 ok($sb->ok(), "Sandbox servers") or BAIL_OUT(__FILE__ . " broke the sandbox");
49+done_testing;
50 exit;

Subscribers

People subscribed via source and target branches