pt-archiver doesn't honor b=1 flag to create SQL_LOG_BIN statement

[maxim.volkov]

Version: percona-toolkit-1.0.1-1

Full command:

pt-archiver \
--source h=localhost,D=database,t=table_I,b=1,i=index \
--dest h=localhost,D=database,t=table_II,b=1,i=index \
--columns column1,column2,column3,column4,column5 \
--where "changed_when < '2011-12-01 00:00:00'" --limit 1000 --commit-each \
--nocheck-columns --replace --commit-each --bulk-insert --bulk-delete \
--statistics \
--no-check-charset \
--file '/tmp/mysql/%Y-%m-%d-%D_%H:%i:%s.%t'

Issue: No ",b=1" nor ",b=true" flags won't create expected SET SQL_LOG_BIN = 0; statement neither with --dry-run nor with actual run.

[Daniel Nichter (daniel-nichter)]

I cannot reproduce this. With 1.0 and 2.1 I see SET SQL_LOG_BIN=0 in the general log when running "pt-archiver --source F=/tmp/12345/my.sandbox.cnf,D=sakila,t=c,b=1 --purge --where 1=1 --no-check-charset".

How are you verifying that SET SQL_LOG_BIN=0 is _not_ being set, Maxim?

[Daniel Nichter (daniel-nichter)]

Here's the relevant code:

      if ( $table->{b} ) {
         $dbh->do("SET SQL_LOG_BIN=0");
      }

So b=<any true value> should work.

[Daniel Nichter (daniel-nichter)]

Maxium, can you reproduce this? See my previous comments.

[maxim.volkov (maxim.volkov)]

Daniel,

The latest version of pt-archiver (2.1) did successfully produced expected result.

Initial verification was by running on a test table, on a master. The table on a master and slave initially was identical. The expected result was - data would be removed on a master and still present on a slave. The actual result - data was removed on both sources.
Running with "--dry-run" didn't produce SET SQL_LOG_BIN=0 either.

[Daniel Nichter (daniel-nichter)]

Thanks Maxim. This is tested now:

ok 26 - Bug 903387: slave has rows
ok 27 - Bug 903387: rows deleted on master
ok 28 - Bug 903387: slave still has rows

[Shahriyar Rzayev (rzayev-sehriyar)]

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PT-910