--recursion-method=dsn does not work in several tools

Reported by William Taylor on 2012-02-17
28
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Percona Toolkit
Medium
Unassigned

Bug Description

It seems like there is partial code in pt-table-sync for this to work but it's not all there

There I get is:
# pt-table-sync --print --recursion-method=dsn=D=percona,t=dsns --replicate percona.checksums h=localhost
Can't locate object method "_find_slaves_by_dsn=D=percona,t=dsns" via package "MasterSlave" at /usr/bin/pt-table-sync line 5977.
Issuing rollback() for database handle being DESTROY'd without explicit disconnect().
Issuing rollback() for database handle being DESTROY'd without explicit disconnect().

pt-table-sync has a get_slaves method like pt-table-checksum but it's not called any where

Can someone connect the dots and make this work ?

In the meantime what is a work around so I can you --replicate and specify a username/password for the slave hosts ?

Daniel Nichter (daniel-nichter) wrote :

On the one hand, this is technically not a bug (and it's a duplicate of bug 933252) because the documentation does not say that "dsn" is a valid value for --recursion-method. On the other hand, since this is the second report and because pt-table-checksum and -sync are so closely related, it seems like a bug to be able to specify slaves via a table for -checksum but not for -sync, which leads to a frustrating and inconsistent user experience. So yes, we'll make --recursion-method=dsn work for pt-table-sync too. As noted in bug 933252, this will involve a bit more than calling another subroutine, but pt-table-sycn is due for an overhaul someday.

Changed in percona-toolkit:
status: New → Confirmed
importance: Undecided → Medium
tags: added: pt-table-sync
Brian Fraser (fraserbn) on 2012-07-30
tags: added: pt-heartbeat pt-slave-find pt-slave-restart
summary: - --recursion-method=dsn does not work in pt-table-sync
+ --recursion-method=dsn does not work in several tools
Daniel Nichter (daniel-nichter) wrote :
Changed in percona-toolkit:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers