Re: max_slot_wal_keep_size unit is not specified

On 2021-Nov-26, Kyotaro Horiguchi wrote:

> Regarding the proposed description, we have never explained how the
> same kind of values specified in megabytes is converted into internal
> units.  If we add that explanation there, we might want the same for
> max_wal_size, min_wal_size and wal_keep_size.

True.  Since this was for backpatch, it seemed best to copy the existing
entries instead of changing them.

> The same section starts with the following sentences.
> 
> >        Specify the maximum size of WAL files
> >        that <link linkend="streaming-replication-slots">replication
> >        slots</link> are allowed to retain in the <filename>pg_wal</filename>
> >        directory at checkpoint time.
> 
> FWIW I thought (or intended) that "the maximum size of WAL files that
> slots are *allowed to retain*" connotes the same to the sentences
> cited above.

Right you are -- that addition was redundant, so I pushed without that.

> Premising "16 MB one way or the other don't change things much", I'm
> not sure we really need to be particular about a few megabites
> difference especially only for this specific variable.

Yeah.

-- 
Álvaro Herrera              Valdivia, Chile  —  https://www.EnterpriseDB.com/