Merge lp:~hrvojem/percona-xtrabackup/bug924313 into lp:percona-xtrabackup/2.0

Proposed by Hrvoje Matijakovic on 2012-02-13
Status: Merged
Approved by: Alexey Kopytov on 2012-02-14
Approved revision: 389
Merged at revision: 391
Proposed branch: lp:~hrvojem/percona-xtrabackup/bug924313
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 82 lines (+8/-6)
5 files modified
doc/source/conf.py (+1/-1)
doc/source/innobackupex/how_innobackupex_works.rst (+1/-1)
doc/source/innobackupex/innobackupex_option_reference.rst (+3/-3)
doc/source/innobackupex/streaming_backups_innobackupex.rst (+2/-0)
src/xtrabackup.c (+1/-1)
To merge this branch: bzr merge lp:~hrvojem/percona-xtrabackup/bug924313
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) 2012-02-13 Approve on 2012-02-14
Review via email: mp+92772@code.launchpad.net
To post a comment you must log in.
Alexey Kopytov (akopytov) wrote :

Looks good, except that the request in bug #924313 was that we make it clear from our documentation that we don't currently support streaming incremental backups. That is, --incremental, --incremental-basedir and --incremental-lsn are ignored when --stream is specified on the innobackupex command line.

I suggest we mention that in the corresponding options descriptions in http://www.percona.com/doc/percona-xtrabackup/innobackupex/innobackupex_option_reference.html and in http://www.percona.com/doc/percona-xtrabackup/innobackupex/streaming_backups_innobackupex.html

Please also not that we are going to fix this in 1.9.1. And that's the first case when the manual will get out of sync with 1.6. I'm not sure what to do about that. We can either create a separate 1.6 manual, or just ignore it and say that the online XtraBackup docs describe only 2.0.

review: Needs Fixing
Stewart Smith (stewart) wrote :

On Mon, 13 Feb 2012 15:32:02 -0000, Alexey Kopytov <email address hidden> wrote:
> Please also not that we are going to fix this in 1.9.1. And that's the
> first case when the manual will get out of sync with 1.6. I'm not sure
> what to do about that. We can either create a separate 1.6 manual, or
> just ignore it and say that the online XtraBackup docs describe only
> 2.0.

This is a good question.... and one that (as you may have noticed) I've
rather successfully delayed :)

My current thought is that we note which parts are applicable only to
2.0 or 1.6 and with the major release following 2.0 we split it off.

--
Stewart Smith

Alexey Kopytov (akopytov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'doc/source/conf.py'
--- doc/source/conf.py 2012-02-08 15:08:50 +0000
+++ doc/source/conf.py 2012-02-14 08:36:17 +0000
@@ -53,7 +53,7 @@
53# The short X.Y version.53# The short X.Y version.
54version = '1.6'54version = '1.6'
55# The full version, including alpha/beta/rc tags.55# The full version, including alpha/beta/rc tags.
56release = '2'56release = '5'
5757
58# The language for content autogenerated by Sphinx. Refer to documentation58# The language for content autogenerated by Sphinx. Refer to documentation
59# for a list of supported languages.59# for a list of supported languages.
6060
=== modified file 'doc/source/innobackupex/how_innobackupex_works.rst'
--- doc/source/innobackupex/how_innobackupex_works.rst 2011-10-24 18:37:06 +0000
+++ doc/source/innobackupex/how_innobackupex_works.rst 2012-02-14 08:36:17 +0000
@@ -53,7 +53,7 @@
53:file:`mysql-stdout`53:file:`mysql-stdout`
54 containing the ``STDOUT`` of the server.54 containing the ``STDOUT`` of the server.
5555
56If the :option:`--remote-host` was set, |innobackupex| will test the connection to the host via :command:`ssh` and create the backup directories. Then the same process will be applied but the log will be written to a temporary file and will be copied via :command:`scp` with the options set by :option:`--options-scp` (``-Cp -c arcfour`` by default).56If the :option:`--remote-host` was set, |innobackupex| will test the connection to the host via :command:`ssh` and create the backup directories. Then the same process will be applied but the log will be written to a temporary file and will be copied via :command:`scp` with the options set by :option:`--scpopt` (``-Cp -c arcfour`` by default).
5757
58After each copy the files will be deleted. The same rationale is for the :option:`--stream` mode.58After each copy the files will be deleted. The same rationale is for the :option:`--stream` mode.
5959
6060
=== modified file 'doc/source/innobackupex/innobackupex_option_reference.rst'
--- doc/source/innobackupex/innobackupex_option_reference.rst 2012-02-02 20:24:24 +0000
+++ doc/source/innobackupex/innobackupex_option_reference.rst 2012-02-14 08:36:17 +0000
@@ -109,7 +109,7 @@
109109
110.. option:: --incremental-lsn110.. option:: --incremental-lsn
111111
112 This option accepts a string argument that specifies the log sequence number (:term:`LSN`) to use for the incremental backup. It is used with the :option:`--incremental` option. It is used instead of specifying :option:`--incremental-basedir`. For databases created by *MySQL* and *Percona Server* 5.0-series versions, specify the as two 32-bit integers in high:low format. For databases created in 5.1 and later, specify the LSN as a single 64-bit integer.112 This option accepts a string argument that specifies the log sequence number (:term:`LSN`) to use for the incremental backup. It is used with the :option:`--incremental` option. It is used instead of specifying :option:`--incremental-basedir`. For databases created by *MySQL* and *Percona Server* 5.0-series versions, specify the as two 32-bit integers in high:low format. For databases created in 5.1 and later, specify the LSN as a single 64-bit integer.
113113
114.. option:: --extra-lsndir114.. option:: --extra-lsndir
115115
@@ -121,7 +121,7 @@
121121
122.. option:: --stream122.. option:: --stream
123123
124 This option accepts a string argument that specifies the format in which to do the streamed backup. The backup will be done to ``STDOUT`` in the specified format. Currently, the only supported format is :command:`tar`. Uses :doc:`tar4ibd <../tar4ibd/tar4ibd_binary>`, which is available in *XtraBackup* distributions. If you specify a path after this option, it will be interpreted as the value of :option:`tmpdir`.124 This option accepts a string argument that specifies the format in which to do the streamed backup. The backup will be done to ``STDOUT`` in the specified format. Currently, the only supported format is :command:`tar`. Uses :doc:`tar4ibd <../tar4ibd/tar4ibd_binary>`, which is available in *XtraBackup* distributions. If you specify a path after this option, it will be interpreted as the value of :option:`tmpdir`. If both :option:`--stream` and :option:`--incremental` are passed to innobackupex, the incremental parameters are ignored and a full backup is created.
125125
126.. option:: --tmpdir126.. option:: --tmpdir
127127
@@ -135,7 +135,7 @@
135135
136 This option forces the use of :command:`tar` when creating a streamed backup, rather than :program:`tar4ibd`, which is the default.136 This option forces the use of :command:`tar` when creating a streamed backup, rather than :program:`tar4ibd`, which is the default.
137137
138.. option:: --scp-opt = '-Cp -c arcfour'138.. option:: --scpopt = '-Cp -c arcfour'
139139
140 This option accepts a string argument that specifies the command line options to pass to :command:`scp` when the option :option:`--remost-host` is specified. If the option is not specified, the default options are ``-Cp -c arcfour``.140 This option accepts a string argument that specifies the command line options to pass to :command:`scp` when the option :option:`--remost-host` is specified. If the option is not specified, the default options are ``-Cp -c arcfour``.
141141
142142
=== modified file 'doc/source/innobackupex/streaming_backups_innobackupex.rst'
--- doc/source/innobackupex/streaming_backups_innobackupex.rst 2011-09-28 01:11:16 +0000
+++ doc/source/innobackupex/streaming_backups_innobackupex.rst 2012-02-14 08:36:17 +0000
@@ -6,6 +6,8 @@
66
7This allows to pipe the stream to other programs, providing great flexibility to the output of it. For example, compression is achieved by piping the output to a compression utility.7This allows to pipe the stream to other programs, providing great flexibility to the output of it. For example, compression is achieved by piping the output to a compression utility.
88
9.. warning:: If both --stream and --incremental are passed to innobackupex, the incremental parameters are ignored and a full backup is created.
10
9To use this feature, you must use the :option:`--stream`, providing the format of the stream (only ``tar`` is supported at this moment) and where should the store the temporary files::11To use this feature, you must use the :option:`--stream`, providing the format of the stream (only ``tar`` is supported at this moment) and where should the store the temporary files::
1012
11 $ innobackupex --stream=tar /tmp13 $ innobackupex --stream=tar /tmp
1214
=== modified file 'src/xtrabackup.c'
--- src/xtrabackup.c 2012-02-10 20:05:56 +0000
+++ src/xtrabackup.c 2012-02-14 08:36:17 +0000
@@ -1036,7 +1036,7 @@
1036 {"extra-lsndir", OPT_XTRA_EXTRA_LSNDIR, "(for --backup): save an extra copy of the xtrabackup_checkpoints file in this directory.",1036 {"extra-lsndir", OPT_XTRA_EXTRA_LSNDIR, "(for --backup): save an extra copy of the xtrabackup_checkpoints file in this directory.",
1037 (G_PTR*) &xtrabackup_extra_lsndir, (G_PTR*) &xtrabackup_extra_lsndir,1037 (G_PTR*) &xtrabackup_extra_lsndir, (G_PTR*) &xtrabackup_extra_lsndir,
1038 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0},1038 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0},
1039 {"incremental-lsn", OPT_XTRA_INCREMENTAL, "(for --backup): copy only .ibd pages newer than specified LSN 'high:low'. ##ATTENTION##: checkpoint lsn must be used. anyone can detect your mistake. be carefully!",1039 {"incremental-lsn", OPT_XTRA_INCREMENTAL, "(for --backup): copy only .ibd pages newer than specified LSN 'high:low'. ##ATTENTION##: If a wrong LSN value is specified, it is impossible to diagnose this, causing the backup to be unusable. Be careful!",
1040 (G_PTR*) &xtrabackup_incremental, (G_PTR*) &xtrabackup_incremental,1040 (G_PTR*) &xtrabackup_incremental, (G_PTR*) &xtrabackup_incremental,
1041 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0},1041 0, GET_STR, REQUIRED_ARG, 0, 0, 0, 0, 0, 0},
1042 {"incremental-basedir", OPT_XTRA_INCREMENTAL_BASEDIR, "(for --backup): copy only .ibd pages newer than backup at specified directory.",1042 {"incremental-basedir", OPT_XTRA_INCREMENTAL_BASEDIR, "(for --backup): copy only .ibd pages newer than backup at specified directory.",

Subscribers

People subscribed via source and target branches