- add --skip-gtid-uuid

- documentation GTID
2025-09-23 12:47:56 +00:00 · 2014-04-30 16:33:27 +02:00
parent bdd969dbdd
commit 5a28a60030
2 changed files with 108 additions and 13 deletions
--- a/bin/pt-slave-restart
+++ b/bin/pt-slave-restart
@@ -5051,9 +5051,17 @@ sub watch_server {
         #    (when a slaveB is replicating from slaveA, 
         #     the master_uuid is it's own master, slaveA)
         my $gtid_exec = $stat->{executed_gtid_set};
-         my $gtid_masteruuid  = $stat->{master_uuid};

-            $gtid_exec        =~ /$gtid_masteruuid([0-9-:]*)/;
+         # default behavior is to take the master_uuid from SHOW SLAVE STATUS
+         # or use --skip-gtid-uuid specified uuid.
+         my $gtid_uuid;
+         if ( $o->get('skip-gtid-uuid') eq 'master' ) {
+            $gtid_uuid = $stat->{master_uuid};
+         } else {
+            $gtid_uuid = $o->get('skip-gtid-uuid');
+         }
+
+            $gtid_exec     =~ /$gtid_uuid([0-9-:]*)/;
         my $gtid_exec_ids =  $1;
            $gtid_exec_ids =~ s/:[0-9]-/:/g;

@@ -5065,10 +5073,10 @@ sub watch_server {
                                grep { defined($_) } @gtid_exec_ranges;
         my $gtid_exec_last   = $gtid_exec_sorted[-1];

-         PTDEBUG && _d("GTID: master_uuid:$gtid_masteruuid,\n"
+         PTDEBUG && _d("GTID: master_uuid:$gtid_uuid,\n"
                     . "GTID: executed_gtid_set:$gtid_exec,\n"
                     . "GTID: gtid max for master_uuid:" . $gtid_exec_sorted[-1] . "\n"
-                     . "GTID: last executed gtid:'$gtid_masteruuid:$gtid_exec_last'");
+                     . "GTID: last executed gtid:'$gtid_uuid:$gtid_exec_last'");

         # Set the sessions next gtid, write an empty transaction
         my $skipped=0;
@@ -5077,10 +5085,10 @@ sub watch_server {

            my $gtid_next=$gtid_exec_last + $skipped;

-            PTDEBUG && _d("GTID: Skipping " . $gtid_masteruuid . ":" . $gtid_next);
+            PTDEBUG && _d("GTID: Skipping " . $gtid_uuid . ":" . $gtid_next);
            
            my $gtid_set_next = $dbh->prepare("SET GTID_NEXT='" 
-                              . $gtid_masteruuid . ":" . $gtid_next . "'");
+                              . $gtid_uuid . ":" . $gtid_next . "'");
            $gtid_set_next->execute();
            $dbh->begin_work();
            $dbh->commit();
@@ -5377,6 +5385,30 @@ sleep time, whichever is less.

 =back

+=head1 GLOBAL TRANSACTION IDS
+
+pt-slave-restart supports Global Transaction IDs, which has been introduced in
+MySQL in 5.6.5.
+
+It's important to keep in mind that:
+
+=over
+
+=item *
+
+pt-slave-restart will not skip transactions when multiple replication threads
+are being used (slave_parallel_workers>0). pt-slave-restart does not know what
+the GTID event is of the failed transaction of a specific slave thread.
+
+=item *
+
+The default behavior is to skip the next transaction from the slave's master.
+Writes can originate on different servers, each with their own unique UUID.
+
+See L<"--skip-gtid-uuid">.
+
+=back
+
 =head1 EXIT STATUS

 An exit status of 0 (sometimes also called a return value or return code)
@@ -5631,6 +5663,23 @@ type: int; default: 1

 Number of statements to skip when restarting the slave.

+=item --skip-gtid-uuid
+
+type: string; default: master
+
+When using GTID, an empty transaction should be created in order to skip it.
+If writes are coming from different nodes in the replication tree above, it is
+not possible to know which event from which UUID to skip.
+
+By default, the UUID from the slave's master is being used to skip. 
+(C<SHOW GLOBAL STATUS Master_UUID> column).
+
+Example: Master -> Slave1 -> Slave2. When skipping events from 'Slave2', and 
+writes originated from 'Master', --skip-gtid-uuid should be specified with the
+'Master' it's UUID.
+
+See L<"GLOBAL TRANSACTION IDS">.
+
 =item --sleep

 type: int; default: 1
--- a/t/pt-slave-restart/gtid.t
+++ b/t/pt-slave-restart/gtid.t
@@ -76,9 +76,9 @@ my $output = `ps -eaf | grep pt-slave-restart | grep -v grep`;
 unlike($output, qr/pt-slave-restart --max/, 'slave: stopped pt-slave-restart successfully');
 diag(`rm -f /tmp/pt-slave-re*`);

-# # #############################################################################
-# # test the slave of the master
-# # #############################################################################
+# #############################################################################
+# test the slave of the master
+# #############################################################################
 $master_dbh->do('DROP DATABASE IF EXISTS test');
 $master_dbh->do('CREATE DATABASE test');
 $master_dbh->do('CREATE TABLE test.t (a INT)');
@@ -94,11 +94,15 @@ wait_until(
   }
 );

+# fetch the master uuid, which is the machine we need to skip an event from
+$r = $master_dbh->selectrow_hashref('select @@GLOBAL.server_uuid as uuid');
+my $uuid = $r->{uuid};
+
 $r = $slave2_dbh->selectrow_hashref('show slave status');
 like($r->{last_error}, qr/Table 'test.t' doesn't exist'/, 'slaveofslave: Replication broke');

 # Start an instance
-diag(`$trunk/bin/pt-slave-restart --max-sleep .25 -h 127.0.0.1 -P 12347 -u msandbox -p msandbox --daemonize --pid /tmp/pt-slave-restart.pid --log /tmp/pt-slave-restart.log`);
+diag(`$trunk/bin/pt-slave-restart --skip-gtid-uuid=$uuid --max-sleep .25 -h 127.0.0.1 -P 12347 -u msandbox -p msandbox --daemonize --pid /tmp/pt-slave-restart.pid --log /tmp/pt-slave-restart.log`);
 sleep 1;

 $r = $slave2_dbh->selectrow_hashref('show slave status');
@@ -110,6 +114,48 @@ sleep 1;
 $output = `ps -eaf | grep pt-slave-restart | grep -v grep`;
 unlike($output, qr/pt-slave-restart --max/, 'slaveofslave: stopped pt-slave-restart successfully');
 diag(`rm -f /tmp/pt-slave-re*`);
+
+
+# #############################################################################
+# test skipping 2 events in a row.
+# #############################################################################
+$master_dbh->do('DROP DATABASE IF EXISTS test');
+$master_dbh->do('CREATE DATABASE test');
+$master_dbh->do('CREATE TABLE test.t (a INT)');
+$sb->wait_for_slaves;
+
+# Bust replication
+$slave2_dbh->do('DROP TABLE test.t');
+$master_dbh->do('INSERT INTO test.t SELECT 1');
+$master_dbh->do('INSERT INTO test.t SELECT 1');
+wait_until(
+   sub {
+      my $row = $slave2_dbh->selectrow_hashref('show slave status');
+      return $row->{last_sql_errno};
+   }
+);
+
+# fetch the master uuid, which is the machine we need to skip an event from
+$r = $master_dbh->selectrow_hashref('select @@GLOBAL.server_uuid as uuid');
+$uuid = $r->{uuid};
+
+$r = $slave2_dbh->selectrow_hashref('show slave status');
+like($r->{last_error}, qr/Table 'test.t' doesn't exist'/, 'slaveofslaveskip2: Replication broke');
+
+# Start an instance
+diag(`$trunk/bin/pt-slave-restart --skip-count=2 --skip-gtid-uuid=$uuid --max-sleep .25 -h 127.0.0.1 -P 12347 -u msandbox -p msandbox --daemonize --pid /tmp/pt-slave-restart.pid --log /tmp/pt-slave-restart.log`);
+sleep 1;
+
+$r = $slave2_dbh->selectrow_hashref('show slave status');
+like($r->{last_errno}, qr/^0$/, 'slaveofslaveskip2: event is not skipped successfully');
+
+
+diag(`$trunk/bin/pt-slave-restart --stop -q`);
+sleep 1;
+$output = `ps -eaf | grep pt-slave-restart | grep -v grep`;
+unlike($output, qr/pt-slave-restart --max/, 'slaveofslaveskip2: stopped pt-slave-restart successfully');
+diag(`rm -f /tmp/pt-slave-re*`);
+
 # #############################################################################
 # Done.
 # #############################################################################