From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Jeff Janes <jeff.janes@gmail.com>
To: David G. Johnston <david.g.johnston@gmail.com>
Cc: pgsql-docs <pgsql-docs@postgresql.org>
Subject: Re: doc bug for continuous archiving
Date: Thu, 8 Oct 2020 13:13:52 +0900
Message-ID: <7c17abbf-4fcf-e8c8-643f-cb33667f33e4@oss.nttdata.com> (raw)
In-Reply-To: <CAMkU=1xdm_CiLGhT5EgciFXakVgh=hdpQDs=w3V76ueu_D_r5w@mail.gmail.com>
References: <CAMkU=1yORNa80450afEYsphadTA2WCRgWf3BYaqgJY14=F4kDg@mail.gmail.com>
<CAKFQuwYj4g7sPev=4ufMzDJEFvbmUgNU_VEqgUcv2noZf7FP=g@mail.gmail.com>
<CAMkU=1xdm_CiLGhT5EgciFXakVgh=hdpQDs=w3V76ueu_D_r5w@mail.gmail.com>
On 2020/10/08 7:12, Jeff Janes wrote:
> On Wed, Oct 7, 2020 at 5:47 PM David G. Johnston <david.g.johnston@gmail.com <mailto:david.g.johnston@gmail.com>> wrote:
>
> On Wed, Oct 7, 2020 at 2:35 PM Jeff Janes <jeff.janes@gmail.com <mailto:jeff.janes@gmail.com>> wrote:
>
> Quoting the docs:
>
> https://www.postgresql.org/docs/13/continuous-archiving.html
>
> "As with base backups, the easiest way to produce a standalone hot backup is to use the pg_basebackup tool. If you include the -X parameter when calling it, all the write-ahead log required to use the backup will be included in the backup automatically, and no special action is required to restore the backup."
>
> This makes it sound like -X is a switch option, when in fact it demands an argument. The statement made is incorrect if the supplied argument is "none", but is correct for the other two options. Should this be changed to spell out which options it applies to (or maybe which option it does not apply to).
>
>
> Just reading the doc page for pg_basebackup: "stream" is the default method value for this option. So if you just include "-X" you get "stream" which is indeed a mode that just works and the continuous archiving section you quote is correct.
>
> I'm not really sure I like the idea that we don't indicate which pg_basebackup key-value argument values are optional and which are mandatory.
>
> David J.
>
> Stream is the default if you don't specify -X at all.
We should use the description like "Unless -X none is specified when calling it" ?
> If you specify -X but with no argument for it, you get an error.
>
> pg_basebackup: option requires an argument -- 'X'
> Try "pg_basebackup --help" for more information.
>
> (Which is also a bit confusing, I would expect a default to apply when the option is specified without an argument, as opposed to when the option itself it is not specified )
As far as I recall, we discussed that in [1] and gave up doing that.
Regards,
[1]
https://postgr.es/m/1338235863.24935.27.camel@vanquo.pezone.net
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: masao.fujii@oss.nttdata.com, jeff.janes@gmail.com, david.g.johnston@gmail.com
Subject: Re: doc bug for continuous archiving
In-Reply-To: <7c17abbf-4fcf-e8c8-643f-cb33667f33e4@oss.nttdata.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox