Merge lp:~akopytov/percona-xtrabackup/bug1168513 into lp:percona-xtrabackup/2.1

Proposed by Alexey Kopytov
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 542
Proposed branch: lp:~akopytov/percona-xtrabackup/bug1168513
Merge into: lp:percona-xtrabackup/2.1
Diff against target: 38 lines (+23/-0)
2 files modified
innobackupex (+5/-0)
test/t/bug1168513.sh (+18/-0)
To merge this branch: bzr merge lp:~akopytov/percona-xtrabackup/bug1168513
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Registry Administrators Pending
Review via email: mp+158746@code.launchpad.net

Description of the change

    Bug #1168513: innobackupex uses SHOW MASTER STATUS to obtain binlog file
                  and position

    Do not create xtrabackup_binlog_info file (and thus do not attempt to
    use undefined values in write_binlog_info()) when no binary log
    information is available.

http://jenkins.percona.com/view/XtraBackup/job/percona-xtrabackup-2.1-param/253/

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'innobackupex'
2--- innobackupex 2013-04-09 10:58:46 +0000
3+++ innobackupex 2013-04-13 08:15:32 +0000
4@@ -1335,6 +1335,11 @@
5 my $filename = $con->{master_status}->{File};
6 my $position = $con->{master_status}->{Position};
7
8+ # Do not create xtrabackup_binlog_info if binary log is disabled
9+ if (!defined($filename) or !defined($position)) {
10+ return;
11+ }
12+
13 # write binlog info file
14 open(FILE, ">$binlog_info") || Die "Failed to open file '$binlog_info': $!";
15 print FILE "$filename\t$position\t\t\n";
16
17=== added file 'test/t/bug1168513.sh'
18--- test/t/bug1168513.sh 1970-01-01 00:00:00 +0000
19+++ test/t/bug1168513.sh 2013-04-13 08:15:32 +0000
20@@ -0,0 +1,18 @@
21+##############################################################################
22+# Bug #1168513: innobackupex uses SHOW MASTER STATUS to obtain binlog file and
23+# position
24+##############################################################################
25+
26+. inc/common.sh
27+
28+# Make sure we start the server with the binary log disabled
29+start_server --skip-log-bin
30+
31+innobackupex --no-timestamp $topdir/backup
32+
33+# Check that to xtrabackup_binlog_info file is created
34+if [ -f $topdir/backup/xtrabackup_binlog_info ]
35+then
36+ vlog "$topdir/backup/xtrabackup_binlog_info should not be created!"
37+ exit -1
38+fi

Subscribers

People subscribed via source and target branches

to all changes: