On Wed, 2019-07-17 at 10:38 +0900, Michael Paquier wrote:
> + <para>
> + Note that while WAL will be flushed with this setting,
> + <application>pg_receivewal</application> never applies it, so
> + <xref linkend="guc-synchronous-commit"/> must not be set to
> + <literal>remote_apply</literal> if <application>pg_receivewal</application>
> + is the only synchronous standby. Similarly, if
> + <application>pg_receivewal</application> is part of a quorum-based
> + set of synchronous standbys, it won't count towards the quorum if
> + <xref linkend="guc-synchronous-commit"/> is set to
> + <literal>remote_apply</literal>.
> + </para>
>
> I think we should really document the caveat with priority-based sets
> of standbys as much as quorum-based sets. For example if a user sets
> synchronous_commit = remote_apply in postgresql.conf, and then sets
> s_s_names to '2(pg_receivewal, my_connected_standby)' to get a
> priority-based set, then you have the same problem, and pg_receivewal
> is not the only synchronous standby in this configuration. The patch
> does not cover that case properly.
I understand the concern, I'm just worried that too much accuracy may
render the sentence hard to read.
How about adding "or priority-based" after "quorum-based"?
Yours,
Laurenz Albe