Merge lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2 into lp:percona-toolkit/2.2

[Daniel Nichter]

[Daniel Nichter (daniel-nichter)]

pt-query-digest/option_sanity...................ok 4/0Cannot open t/pt-query-digest/samples/stats-slow002.txt: No such file or directory at /Users/daniel/p/clean-up-pqd-2.2/lib/PerconaTest.pm line 213.

review: Needs Fixing

[lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2]

539. By Daniel Nichter on 2013-03-05

Clean up docs. Clean up t/pt-query-digest/since_until.t. review.t needs to be fixed and cleaned up (i.e. use pt_query_digest::main() instead of shelling out, split off --history tests into history.t, etc.).

540. By Daniel Nichter on 2013-03-05

Merge Brian's change in r539.

[Daniel Nichter (daniel-nichter)]

review.t needs to be fixed and cleaned up.

review: Needs Fixing

[lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2]

541. By Daniel Nichter on 2013-03-05

Remove --zero-admin; always on now. Change --output=query to =report and update the docs.

542. By Brian Fraser on 2013-03-06

Make review.t use output() rather than shelling out

543. By Brian Fraser on 2013-03-06

Move the --history tests out of review.t

544. By Brian Fraser on 2013-03-06

Fix broken tests

[Daniel Nichter (daniel-nichter)]

http://bazaar.launchpad.net/~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2/revision/544

The --[no]zero-admin optino was removed but its affect kept in place, i.e. made always-on, so we should keep the test that verifies that Admin commands have zero values for Row* attributes.

review: Needs Fixing

[Daniel Nichter (daniel-nichter)]

Cancel that; my mistake. I read it backwards. I see now that the removed test was for --no-zero-admin.

> http://bazaar.launchpad.net/~percona-toolkit-dev/percona-toolkit/clean-up-
> pqd-2.2/revision/544
>
> The --[no]zero-admin optino was removed but its affect kept in place, i.e.
> made always-on, so we should keep the test that verifies that Admin commands
> have zero values for Row* attributes.

[lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2]

545. By Daniel Nichter on 2013-03-06

Rename hist.t to history.t.

[Daniel Nichter (daniel-nichter)]

Something went wrong: https://refute.testnoir.com/percona-toolkit/jobs/percona-toolkit-all-platforms/71/results/errors

review: Needs Fixing

[lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2]

546. By Brian Fraser on 2013-03-07

Split the QueryHistory tests out of QueryReview.t

[Daniel Nichter (daniel-nichter)]

Tested and working: https://refute.testnoir.com/percona-toolkit/jobs/percona-toolkit-full-branch-test/22/results/skipped

review: Approve

[lp:~percona-toolkit-dev/percona-toolkit/clean-up-pqd-2.2]

547. By Daniel Nichter on 2013-03-09

Clean up and fix docs again.

548. By Daniel Nichter on 2013-03-09

Note that json output is in development. Fix typo.