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
1=== modified file 'innobackupex'
2--- innobackupex 2011-04-29 14:12:21 +0000
3+++ innobackupex 2011-05-20 17:30:29 +0000
4@@ -361,39 +361,35 @@
5 wait_for_ibbackup_suspend();
6 }
7
8- if (!$option_incremental) {
9- # connect to database
10- mysql_open();
11-
12- if ( $option_safe_slave_backup ) {
13- wait_for_safe_slave();
14- }
15-
16- # flush tables with read lock
17- mysql_check();
18- mysql_lockall() if !$option_no_lock;
19-
20+ # connect to database
21+ mysql_open();
22+
23+ if ( $option_safe_slave_backup ) {
24+ wait_for_safe_slave();
25 }
26
27+ # flush tables with read lock
28+ mysql_check();
29+ mysql_lockall() if !$option_no_lock;
30+
31+
32 # backup .frm, .MRG, .MYD, .MYI, .TRG, .TRN, .ARM, .ARZ, .CSM, .CSV and .opt files
33 backup_files();
34
35 # resume ibbackup and wait till it has finished
36 my $ibbackup_exit_code = resume_ibbackup();
37
38- if (!$option_incremental) {
39- # release read locks on all tables
40- mysql_unlockall() if !$option_no_lock;
41-
42- if ( $option_safe_slave_backup ) {
43- print STDERR "$prefix: Starting slave SQL thread\n";
44- mysql_send('START SLAVE SQL_THREAD;');
45- }
46-
47- # disconnect from database
48- mysql_close();
49+ # release read locks on all tables
50+ mysql_unlockall() if !$option_no_lock;
51+
52+ if ( $option_safe_slave_backup ) {
53+ print STDERR "$prefix: Starting slave SQL thread\n";
54+ mysql_send('START SLAVE SQL_THREAD;');
55 }
56
57+ # disconnect from database
58+ mysql_close();
59+
60 if ($option_remote_host) {
61 system("scp $option_scp_opt '$tmp_logfile' '$option_remote_host:$backup_dir/xtrabackup_logfile'")
62 and Die "Failed to scp file '$option_remote_host:$backup_dir/xtrabackup_logfile': $!";

Subscribers

People subscribed via source and target branches