Re: fixes for a few GUC descriptions

On 2026-Sep-11, Chao Li wrote:

> > On Sep 11, 2026, at 08:54, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> wrote:

> > - long_desc => '-1 disables the timeout and waits for the receiver to
> > catch up; 0 does not wait for the receiver to catch up.',
> > + long_desc => '-1 disables the timeout. 0 means do not wait for the
> > receiver to catch up.',

>
> Yeah, here -1 is really more like “wait forever”, so just saying “disables the timeout” feels like it loses that distinction.

Maybe it should say "0 means do not wait, -1 means wait indefinitely."
This complements the short_desc,

Sets the maximum time the server waits during shutdown for all WAL data
to be replicated to the receiver.

I think the words "for the receiver to catch up" is somehow at odds with
the short_desc's "for all WAL data to be replicated". They should be
saying the same thing, but aren't.

--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
Si no sabes adonde vas, es muy probable que acabes en otra parte.