percona/percona-toolkit
1
0
Fork 0
mirror of https://github.com/percona/percona-toolkit.git synced 2026-04-17 01:01:39 +08:00
Code Issues Packages Projects Releases Wiki Activity

Turn off statement based binlog checks

Browse Source 
For RDS we use row based replication however `pt-table-checksum` wants
to enforce statement and despite passing in the
`--no-check-binlog-format` flag, it would still run it (even though the
documentation suggests[1] it shouldn't). This appears to be a bug
because there is another binlog format check that is wrapped in
`$o->get('check-binlog-format')` slightly further down.

To resolve this, I've updated the remaining binlog format check in the
same expression as the one below and it now honours the
`--no-check-binlog-format` as expected.

[1]: https://www.percona.com/doc/percona-toolkit/3.0/pt-table-checksum.html#cmdoption-pt-table-checksum--[no]check-binlog-format
This commit is contained in:
Jacob Bednarz
2017-03-24 08:31:25 +11:00
parent 7b0d02b8b8
commit 0271ba6a09
1 changed files with 29 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

57
bin/pt-table-checksum
View File

@@ -9272,35 +9272,36 @@ sub main {
die "Error setting SQL_MODE" die "Error setting SQL_MODE"
. ": $EVAL_ERROR"; . ": $EVAL_ERROR";
} }
# https://bugs.launchpad.net/percona-toolkit/+bug/919352 if ( $o->get('check-binlog-format') ) {
# The tool shouldn't blindly attempt to change binlog_format; # https://bugs.launchpad.net/percona-toolkit/+bug/919352
# instead, it should check if it's already set to STATEMENT. # The tool shouldn't blindly attempt to change binlog_format;
# This is becase starting with MySQL 5.1.29, changing the format # instead, it should check if it's already set to STATEMENT.
# requires a SUPER user. # This is becase starting with MySQL 5.1.29, changing the format
if ( VersionParser->new($dbh) >= '5.1.5' ) { # requires a SUPER user.
$sql = 'SELECT @@binlog_format'; if ( VersionParser->new($dbh) >= '5.1.5' ) {
PTDEBUG && _d($dbh, $sql); $sql = 'SELECT @@binlog_format';
my ($original_binlog_format) = $dbh->selectrow_array($sql); PTDEBUG && _d($dbh, $sql);
PTDEBUG && _d('Original binlog_format:', $original_binlog_format); my ($original_binlog_format) = $dbh->selectrow_array($sql);
if ( $original_binlog_format !~ /STATEMENT/i ) { PTDEBUG && _d('Original binlog_format:', $original_binlog_format);
$sql = q{/*!50108 SET @@binlog_format := 'STATEMENT'*/}; if ( $original_binlog_format !~ /STATEMENT/i ) {
eval { $sql = q{/*!50108 SET @@binlog_format := 'STATEMENT'*/};
PTDEBUG && _d($dbh, $sql); eval {
$dbh->do($sql); PTDEBUG && _d($dbh, $sql);
}; $dbh->do($sql);
if ( $EVAL_ERROR ) { };
die "Failed to $sql: $EVAL_ERROR\n" if ( $EVAL_ERROR ) {
. "This tool requires binlog_format=STATEMENT, " die "Failed to $sql: $EVAL_ERROR\n"
. "but the current binlog_format is set to " . "This tool requires binlog_format=STATEMENT, "
."$original_binlog_format and an error occurred while " . "but the current binlog_format is set to "
. "attempting to change it. If running MySQL 5.1.29 or newer, " ."$original_binlog_format and an error occurred while "
. "setting binlog_format requires the SUPER privilege. " . "attempting to change it. If running MySQL 5.1.29 or newer, "
. "You will need to manually set binlog_format to 'STATEMENT' " . "setting binlog_format requires the SUPER privilege. "
. "before running this tool.\n"; . "You will need to manually set binlog_format to 'STATEMENT' "
} . "before running this tool.\n";
} }
}
}
} }
# Set transaction isolation level. We set binlog_format to STATEMENT, # Set transaction isolation level. We set binlog_format to STATEMENT,