Merge lp:~percona-dev/percona-xtrabackup/bugfix759701 into lp:percona-xtrabackup/2.0

Proposed by Vadim Tkachenko
Status: Merged
Approved by: Valentine Gostev
Approved revision: no longer in the source branch.
Merged at revision: 257
Proposed branch: lp:~percona-dev/percona-xtrabackup/bugfix759701
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 62 lines (+19/-23)
1 file modified
innobackupex (+19/-23)
To merge this branch: bzr merge lp:~percona-dev/percona-xtrabackup/bugfix759701
Reviewer Review Type Date Requested Status
Valentine Gostev (community) qa Approve
Review via email: mp+61805@code.launchpad.net

Description of the change

This supposes to fix
https://bugs.launchpad.net/percona-xtrabackup/+bug/771981
https://bugs.launchpad.net/percona-xtrabackup/+bug/783596

This removes condition in innobackupex that non-InnoDB tables and files
are not handled in incremental.

This fix should be throughly tested in innobackupex + incremental schema.

To post a comment you must log in.
Revision history for this message
Valentine Gostev (longbow) wrote :

Looks good.
Keeping open connection resolved mentioned problems.

review: Approve (qa)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'innobackupex'
--- innobackupex 2011-04-29 14:12:21 +0000
+++ innobackupex 2011-05-20 17:30:29 +0000
@@ -361,39 +361,35 @@
361 wait_for_ibbackup_suspend();361 wait_for_ibbackup_suspend();
362 }362 }
363363
364 if (!$option_incremental) {364 # connect to database
365 # connect to database365 mysql_open();
366 mysql_open();366
367367 if ( $option_safe_slave_backup ) {
368 if ( $option_safe_slave_backup ) {368 wait_for_safe_slave();
369 wait_for_safe_slave();
370 }
371
372 # flush tables with read lock
373 mysql_check();
374 mysql_lockall() if !$option_no_lock;
375
376 }369 }
377370
371 # flush tables with read lock
372 mysql_check();
373 mysql_lockall() if !$option_no_lock;
374
375
378 # backup .frm, .MRG, .MYD, .MYI, .TRG, .TRN, .ARM, .ARZ, .CSM, .CSV and .opt files376 # backup .frm, .MRG, .MYD, .MYI, .TRG, .TRN, .ARM, .ARZ, .CSM, .CSV and .opt files
379 backup_files();377 backup_files();
380378
381 # resume ibbackup and wait till it has finished379 # resume ibbackup and wait till it has finished
382 my $ibbackup_exit_code = resume_ibbackup();380 my $ibbackup_exit_code = resume_ibbackup();
383381
384 if (!$option_incremental) {382 # release read locks on all tables
385 # release read locks on all tables383 mysql_unlockall() if !$option_no_lock;
386 mysql_unlockall() if !$option_no_lock;384
387385 if ( $option_safe_slave_backup ) {
388 if ( $option_safe_slave_backup ) {386 print STDERR "$prefix: Starting slave SQL thread\n";
389 print STDERR "$prefix: Starting slave SQL thread\n";387 mysql_send('START SLAVE SQL_THREAD;');
390 mysql_send('START SLAVE SQL_THREAD;');
391 }
392
393 # disconnect from database
394 mysql_close();
395 }388 }
396389
390 # disconnect from database
391 mysql_close();
392
397 if ($option_remote_host) {393 if ($option_remote_host) {
398 system("scp $option_scp_opt '$tmp_logfile' '$option_remote_host:$backup_dir/xtrabackup_logfile'")394 system("scp $option_scp_opt '$tmp_logfile' '$option_remote_host:$backup_dir/xtrabackup_logfile'")
399 and Die "Failed to scp file '$option_remote_host:$backup_dir/xtrabackup_logfile': $!";395 and Die "Failed to scp file '$option_remote_host:$backup_dir/xtrabackup_logfile': $!";

Subscribers

People subscribed via source and target branches