pg.ddx.io pgsql-committers@postgresql.org mailing list archive
help / color / mirror / Atom feedpgsql: doc: Further clarify how recovery target parameters are applied
12+ messages / 5 participants
[nested] [flat]
Commits from this thread:
2019-11-09 27b59d619d 175571923c doc: Further tweak recovery parameters documentation
* pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-09-29 21:10 Peter Eisentraut <peter@eisentraut.org>
0 siblings, 1 reply; 12+ messages in thread
From: Peter Eisentraut @ 2019-09-29 21:10 UTC (permalink / raw)
To: pgsql-committers@lists.postgresql.org
doc: Further clarify how recovery target parameters are applied
Recovery target parameters are all applied even in standby mode. The
previous documentation mostly wished they were not but this was never
the case.
Discussion: https://www.postgresql.org/message-id/flat/e445616d-023e-a268-8aa1-67b8b335340c%40pgmasters.net
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/e04a53a6071d13ef4a13a41c6419d8e14c30b95c
Modified Files
--------------
doc/src/sgml/config.sgml | 20 +++++++++-----------
1 file changed, 9 insertions(+), 11 deletions(-)
^ permalink raw reply [nested|flat] 12+ messages in thread
* pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-09-29 21:10 Peter Eisentraut <peter@eisentraut.org>
0 siblings, 0 replies; 12+ messages in thread
From: Peter Eisentraut @ 2019-09-29 21:10 UTC (permalink / raw)
To: pgsql-committers@lists.postgresql.org
doc: Further clarify how recovery target parameters are applied
Recovery target parameters are all applied even in standby mode. The
previous documentation mostly wished they were not but this was never
the case.
Discussion: https://www.postgresql.org/message-id/flat/e445616d-023e-a268-8aa1-67b8b335340c%40pgmasters.net
Branch
------
REL_12_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/a7e5ae53cbe7bd88102b38d5f6e15f5adc94e15c
Modified Files
--------------
doc/src/sgml/config.sgml | 20 +++++++++-----------
1 file changed, 9 insertions(+), 11 deletions(-)
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-09-30 13:14 Fujii Masao <masao.fujii@gmail.com>
parent: Peter Eisentraut <peter@eisentraut.org>
0 siblings, 1 reply; 12+ messages in thread
From: Fujii Masao @ 2019-09-30 13:14 UTC (permalink / raw)
To: Peter Eisentraut <peter@eisentraut.org>; +Cc: pgsql-committers@lists.postgresql.org
On Mon, Sep 30, 2019 at 6:10 AM Peter Eisentraut <peter@eisentraut.org> wrote:
>
> doc: Further clarify how recovery target parameters are applied
>
> Recovery target parameters are all applied even in standby mode. The
> previous documentation mostly wished they were not but this was never
> the case.
Thanks for improving the documentation.
+ In this mode, the parameters from both this section and <xref
+ linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
+ from <xref linkend="runtime-config-replication-standby"/> will not be
+ used.
The latter description seems not true. hot_standby and
max_standby_archive_delay in "runtime-config-replication-standby"
are used during archive recovery. So something like the following
change would need to be applied, I think. Patch attached.
In this mode, the parameters from both this section and <xref
linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
from <xref linkend="runtime-config-replication-standby"/> will not be
- used.
+ used (<varname>hot_standby</varname> and
+ <varname>max_standby_archive_delay</varname>).
</para>
Regards,
--
Fujii Masao
Attachments:
standby_parameters_document_v1.patch (application/octet-stream, 576B) download
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 6f55a2b5c9..77d276edfe 100644
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -3169,7 +3169,8 @@ include_dir 'conf.d'
In this mode, the parameters from both this section and <xref
linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
from <xref linkend="runtime-config-replication-standby"/> will not be
- used.
+ used (<varname>hot_standby</varname> and
+ <varname>max_standby_archive_delay</varname>).
</para>
<variablelist>
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-10-18 14:18 Fujii Masao <masao.fujii@gmail.com>
parent: Fujii Masao <masao.fujii@gmail.com>
0 siblings, 1 reply; 12+ messages in thread
From: Fujii Masao @ 2019-10-18 14:18 UTC (permalink / raw)
To: Peter Eisentraut <peter@eisentraut.org>; +Cc: pgsql-committers@lists.postgresql.org
On Mon, Sep 30, 2019 at 10:14 PM Fujii Masao <masao.fujii@gmail.com> wrote:
>
> On Mon, Sep 30, 2019 at 6:10 AM Peter Eisentraut <peter@eisentraut.org> wrote:
> >
> > doc: Further clarify how recovery target parameters are applied
> >
> > Recovery target parameters are all applied even in standby mode. The
> > previous documentation mostly wished they were not but this was never
> > the case.
>
> Thanks for improving the documentation.
>
> + In this mode, the parameters from both this section and <xref
> + linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
> + from <xref linkend="runtime-config-replication-standby"/> will not be
> + used.
>
> The latter description seems not true. hot_standby and
> max_standby_archive_delay in "runtime-config-replication-standby"
> are used during archive recovery. So something like the following
> change would need to be applied, I think. Patch attached.
>
> In this mode, the parameters from both this section and <xref
> linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
> from <xref linkend="runtime-config-replication-standby"/> will not be
> - used.
> + used (<varname>hot_standby</varname> and
> + <varname>max_standby_archive_delay</varname>).
> </para>
In the above change, I forgot to add "except" before "<varname>hot_standby"...
Now, not only hot_standby and max_standby_archive_delay but also
recovery_min_apply_delay from
<xref linkend="runtime-config-replication-standby"/> can be used in
archive recovery, thanks to commit ec1259e880. So I'm just inclined to
remove the description "Parameters from
<xref linkend="runtime-config-replication-standby"/> will not be used."
rather than to list all "exceptive" parameters.
Regards,
--
Fujii Masao
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-05 13:21 Alvaro Herrera <alvherre@2ndquadrant.com>
parent: Fujii Masao <masao.fujii@gmail.com>
0 siblings, 1 reply; 12+ messages in thread
From: Alvaro Herrera @ 2019-11-05 13:21 UTC (permalink / raw)
To: Fujii Masao <masao.fujii@gmail.com>; +Cc: Peter Eisentraut <peter@eisentraut.org>; pgsql-committers@lists.postgresql.org
On 2019-Oct-18, Fujii Masao wrote:
> > Thanks for improving the documentation.
> >
> > + In this mode, the parameters from both this section and <xref
> > + linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
> > + from <xref linkend="runtime-config-replication-standby"/> will not be
> > + used.
> >
> > The latter description seems not true. hot_standby and
> > max_standby_archive_delay in "runtime-config-replication-standby"
> > are used during archive recovery. So something like the following
> > change would need to be applied, I think. Patch attached.
This has not been applied, right? Can you send a complete patch?
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-06 04:48 Fujii Masao <masao.fujii@gmail.com>
parent: Alvaro Herrera <alvherre@2ndquadrant.com>
0 siblings, 1 reply; 12+ messages in thread
From: Fujii Masao @ 2019-11-06 04:48 UTC (permalink / raw)
To: Alvaro Herrera <alvherre@2ndquadrant.com>; +Cc: Peter Eisentraut <peter@eisentraut.org>; pgsql-committers@lists.postgresql.org
On Tue, Nov 5, 2019 at 10:28 PM Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>
> On 2019-Oct-18, Fujii Masao wrote:
>
> > > Thanks for improving the documentation.
> > >
> > > + In this mode, the parameters from both this section and <xref
> > > + linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
> > > + from <xref linkend="runtime-config-replication-standby"/> will not be
> > > + used.
> > >
> > > The latter description seems not true. hot_standby and
> > > max_standby_archive_delay in "runtime-config-replication-standby"
> > > are used during archive recovery. So something like the following
> > > change would need to be applied, I think. Patch attached.
>
> This has not been applied, right?
Yes.
> Can you send a complete patch?
Thanks for the ping!
Patch attached. As I argued upthread, IMO it's better to remove
the latter description from the doc and the patch does that.
Also the patch adds "mainly" into the former description.
Regards,
--
Fujii Masao
Attachments:
doc_guc_for_standby.patch (application/octet-stream, 1011B) download
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 0191ec84b1..37e66add44 100644
*** a/doc/src/sgml/config.sgml
--- b/doc/src/sgml/config.sgml
***************
*** 3217,3225 **** include_dir 'conf.d'
takes precedence. Targeted recovery mode ends when the archived WAL is
fully replayed, or when <varname>recovery_target</varname> is reached.
In this mode, the parameters from both this section and <xref
! linkend="runtime-config-wal-recovery-target"/> will be used. Parameters
! from <xref linkend="runtime-config-replication-standby"/> will not be
! used.
</para>
<variablelist>
--- 3217,3223 ----
takes precedence. Targeted recovery mode ends when the archived WAL is
fully replayed, or when <varname>recovery_target</varname> is reached.
In this mode, the parameters from both this section and <xref
! linkend="runtime-config-wal-recovery-target"/> will be mainly used.
</para>
<variablelist>
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-06 07:29 Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
parent: Fujii Masao <masao.fujii@gmail.com>
0 siblings, 1 reply; 12+ messages in thread
From: Peter Eisentraut @ 2019-11-06 07:29 UTC (permalink / raw)
To: Fujii Masao <masao.fujii@gmail.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; +Cc: pgsql-committers@lists.postgresql.org
On 2019-11-06 05:48, Fujii Masao wrote:
> Patch attached. As I argued upthread, IMO it's better to remove
> the latter description from the doc and the patch does that.
> Also the patch adds "mainly" into the former description.
I think we should list explicitly what is applied and what is not. This
is the reference documentation after all.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-06 09:49 Fujii Masao <masao.fujii@gmail.com>
parent: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
0 siblings, 2 replies; 12+ messages in thread
From: Fujii Masao @ 2019-11-06 09:49 UTC (permalink / raw)
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-committers@lists.postgresql.org
On Wed, Nov 6, 2019 at 4:29 PM Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
>
> On 2019-11-06 05:48, Fujii Masao wrote:
> > Patch attached. As I argued upthread, IMO it's better to remove
> > the latter description from the doc and the patch does that.
> > Also the patch adds "mainly" into the former description.
>
> I think we should list explicitly what is applied and what is not. This
> is the reference documentation after all.
That's possible, but may make the document more complicated and confusing,
I'm afraid. Because the parameters used for archive recovery and standby mode
are not categorized cleanly. For example,
- Parameters from Section 19.6.1. "Sending Servers" will be used except
track_commit_timestamp (in standby mode) and except track_commit_timestamp
and wal_init_zero (in archive recovery)
- Parameters from Section 19.6.3. "Standby Servers" will be used in standby
mode, but will not used except hot_standby, recovery_min_apply_delay, etc
in archive recovery.
- Parameters from Section 19.5.3. "Archiving" will be used in both standby mode
and archive recovery, but except archive_timeout.
So we would need to list such several parameters like the above
and which might make the document more complicated. Also which would
make the maintenance of the document harder.
Or it might be better to add something like "This setting has no effect if
in standby mode / archive recovery" into the description of each parameter,
instead.
Regards,
--
Fujii Masao
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-07 15:23 Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
parent: Fujii Masao <masao.fujii@gmail.com>
1 sibling, 1 reply; 12+ messages in thread
From: Peter Eisentraut @ 2019-11-07 15:23 UTC (permalink / raw)
To: Fujii Masao <masao.fujii@gmail.com>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-committers@lists.postgresql.org
On 2019-11-06 10:49, Fujii Masao wrote:
> On Wed, Nov 6, 2019 at 4:29 PM Peter Eisentraut
> <peter.eisentraut@2ndquadrant.com> wrote:
>>
>> On 2019-11-06 05:48, Fujii Masao wrote:
>>> Patch attached. As I argued upthread, IMO it's better to remove
>>> the latter description from the doc and the patch does that.
>>> Also the patch adds "mainly" into the former description.
>>
>> I think we should list explicitly what is applied and what is not. This
>> is the reference documentation after all.
>
> That's possible, but may make the document more complicated and confusing,
> I'm afraid. Because the parameters used for archive recovery and standby mode
> are not categorized cleanly.
After reading this again and again, I think the best fix to get PG12
into "correct" shape is to just remove the offending sentence:
inline patch 1: doc/src/sgml/config.sgml (+0/-0 lines) download inline-1.patch
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 46bc31de4c..fdebe9bf4b 100644
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -3217,9 +3217,7 @@ <title>Archive Recovery</title>
takes precedence. Targeted recovery mode ends when the archived
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-07 17:10 Fujii Masao <masao.fujii@gmail.com>
parent: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
0 siblings, 1 reply; 12+ messages in thread
From: Fujii Masao @ 2019-11-07 17:10 UTC (permalink / raw)
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-committers@lists.postgresql.org
On Fri, Nov 8, 2019 at 12:23 AM Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
>
> On 2019-11-06 10:49, Fujii Masao wrote:
> > On Wed, Nov 6, 2019 at 4:29 PM Peter Eisentraut
> > <peter.eisentraut@2ndquadrant.com> wrote:
> >>
> >> On 2019-11-06 05:48, Fujii Masao wrote:
> >>> Patch attached. As I argued upthread, IMO it's better to remove
> >>> the latter description from the doc and the patch does that.
> >>> Also the patch adds "mainly" into the former description.
> >>
> >> I think we should list explicitly what is applied and what is not. This
> >> is the reference documentation after all.
> >
> > That's possible, but may make the document more complicated and confusing,
> > I'm afraid. Because the parameters used for archive recovery and standby mode
> > are not categorized cleanly.
>
> After reading this again and again, I think the best fix to get PG12
> into "correct" shape is to just remove the offending sentence:
+1
> As we are discussing in other threads, there is further opportunity to
> reorganize how these settings interact, but that's a different project.
Yes.
Regards,
--
Fujii Masao
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-07 17:33 David Steele <david@pgmasters.net>
parent: Fujii Masao <masao.fujii@gmail.com>
1 sibling, 0 replies; 12+ messages in thread
From: David Steele @ 2019-11-07 17:33 UTC (permalink / raw)
To: Fujii Masao <masao.fujii@gmail.com>; Peter Eisentraut <peter.eisentraut@2ndquadrant.com>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-committers@lists.postgresql.org
On 11/6/19 4:49 AM, Fujii Masao wrote:
> On Wed, Nov 6, 2019 at 4:29 PM Peter Eisentraut
> <peter.eisentraut@2ndquadrant.com> wrote:
>>
>> On 2019-11-06 05:48, Fujii Masao wrote:
>>> Patch attached. As I argued upthread, IMO it's better to remove
>>> the latter description from the doc and the patch does that.
>>> Also the patch adds "mainly" into the former description.
>>
>> I think we should list explicitly what is applied and what is not. This
>> is the reference documentation after all.
>
> Or it might be better to add something like "This setting has no effect if
> in standby mode / archive recovery" into the description of each parameter,
> instead.
+1. This will probably be easier to maintain and less confusing overall.
--
-David
david@pgmasters.net
^ permalink raw reply [nested|flat] 12+ messages in thread
* Re: pgsql: doc: Further clarify how recovery target parameters are applied
@ 2019-11-09 08:39 Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
parent: Fujii Masao <masao.fujii@gmail.com>
0 siblings, 0 replies; 12+ messages in thread
From: Peter Eisentraut @ 2019-11-09 08:39 UTC (permalink / raw)
To: Fujii Masao <masao.fujii@gmail.com>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-committers@lists.postgresql.org
On 2019-11-07 18:10, Fujii Masao wrote:
>> After reading this again and again, I think the best fix to get PG12
>> into "correct" shape is to just remove the offending sentence:
>
> +1
done
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 12+ messages in thread
end of thread, other threads:[~2019-11-09 08:39 UTC | newest]
Thread overview: 12+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2019-09-29 21:10 pgsql: doc: Further clarify how recovery target parameters are applied Peter Eisentraut <peter@eisentraut.org>
2019-09-30 13:14 ` Fujii Masao <masao.fujii@gmail.com>
2019-10-18 14:18 ` Fujii Masao <masao.fujii@gmail.com>
2019-11-05 13:21 ` Alvaro Herrera <alvherre@2ndquadrant.com>
2019-11-06 04:48 ` Fujii Masao <masao.fujii@gmail.com>
2019-11-06 07:29 ` Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
2019-11-06 09:49 ` Fujii Masao <masao.fujii@gmail.com>
2019-11-07 15:23 ` Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
2019-11-07 17:10 ` Fujii Masao <masao.fujii@gmail.com>
2019-11-09 08:39 ` Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
2019-11-07 17:33 ` David Steele <david@pgmasters.net>
2019-09-29 21:10 pgsql: doc: Further clarify how recovery target parameters are applied Peter Eisentraut <peter@eisentraut.org>
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