Merge lp:~sergei.glushchenko/percona-xtrabackup/2.2-xb-bug1347698 into lp:percona-xtrabackup/2.2

Proposed by Sergei Glushchenko
Status: Merged
Approved by: Alexey Kopytov
Approved revision: no longer in the source branch.
Merged at revision: 5003
Proposed branch: lp:~sergei.glushchenko/percona-xtrabackup/2.2-xb-bug1347698
Merge into: lp:percona-xtrabackup/2.2
Diff against target: 56 lines (+27/-0)
2 files modified
storage/innobase/xtrabackup/src/xtrabackup.cc (+8/-0)
storage/innobase/xtrabackup/test/t/bug1347698.sh (+19/-0)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-xtrabackup/2.2-xb-bug1347698
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Approve
Review via email: mp+233642@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :
Revision history for this message
Alexey Kopytov (akopytov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'storage/innobase/xtrabackup/src/xtrabackup.cc'
2--- storage/innobase/xtrabackup/src/xtrabackup.cc 2014-09-04 16:53:33 +0000
3+++ storage/innobase/xtrabackup/src/xtrabackup.cc 2014-09-09 15:38:32 +0000
4@@ -199,6 +199,9 @@
5 in milliseconds (default is 1 second) */
6 ulint xtrabackup_log_copy_interval = 1000;
7
8+/* Ignored option (--log) for MySQL option compatibility */
9+char* log_ignored_opt = NULL;
10+
11 /* === metadata of backup === */
12 #define XTRABACKUP_METADATA_FILENAME "xtrabackup_checkpoints"
13 char metadata_type[30] = ""; /*[full-backuped|full-prepared|incremental]*/
14@@ -426,6 +429,7 @@
15 OPT_XTRA_ENCRYPT_KEY_FILE,
16 OPT_XTRA_ENCRYPT_THREADS,
17 OPT_XTRA_ENCRYPT_CHUNK_SIZE,
18+ OPT_LOG,
19 OPT_INNODB,
20 OPT_INNODB_CHECKSUMS,
21 OPT_INNODB_DATA_FILE_PATH,
22@@ -630,6 +634,10 @@
23 (G_PTR*) &xtrabackup_encrypt_chunk_size, (G_PTR*) &xtrabackup_encrypt_chunk_size,
24 0, GET_ULL, REQUIRED_ARG, (1 << 16), 1024, ULONGLONG_MAX, 0, 0, 0},
25
26+ {"log", OPT_LOG, "Ignored option for MySQL option compatibility",
27+ (G_PTR*) &log_ignored_opt, (G_PTR*) &log_ignored_opt, 0,
28+ GET_STR, OPT_ARG, 0, 0, 0, 0, 0, 0},
29+
30 {"innodb", OPT_INNODB, "Ignored option for MySQL option compatibility",
31 (G_PTR*) &innobase_ignored_opt, (G_PTR*) &innobase_ignored_opt, 0,
32 GET_STR, OPT_ARG, 0, 0, 0, 0, 0, 0},
33
34=== added file 'storage/innobase/xtrabackup/test/t/bug1347698.sh'
35--- storage/innobase/xtrabackup/test/t/bug1347698.sh 1970-01-01 00:00:00 +0000
36+++ storage/innobase/xtrabackup/test/t/bug1347698.sh 2014-09-09 15:38:32 +0000
37@@ -0,0 +1,19 @@
38+########################################################################
39+# Bug #1347698: log parameter on mysqld section shouldn't be read by
40+# xtrabackup
41+########################################################################
42+
43+. inc/common.sh
44+
45+require_server_version_lower_than 5.6.0
46+
47+# add --log parameter to [mysqld] section of my.cnf
48+MYSQLD_EXTRA_MY_CNF_OPTS="
49+log=$MYSQLD_VARDIR/general.log
50+"
51+
52+start_server
53+
54+vlog "Creating the backup directory: $topdir/backup"
55+mkdir -p $topdir/backup
56+xtrabackup --backup --target-dir=$topdir/backup

Subscribers

People subscribed via source and target branches