From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: pgsql-docs@lists.postgresql.org
Subject: Re: The description for pg_replication_slots.restart_lsn
Date: Tue, 30 Jun 2020 14:56:50 +0900
Message-ID: <b495e29a-4131-81f6-7d0d-dcbb51cc5152@oss.nttdata.com> (raw)
In-Reply-To: <0414a23a-974c-04d5-ccb0-70ebcc76fcb7@oss.nttdata.com>
References: <20200625010016.GA22779@alvherre.pgsql>
<0414a23a-974c-04d5-ccb0-70ebcc76fcb7@oss.nttdata.com>
On 2020/06/25 14:48, Fujii Masao wrote:
>
>
> On 2020/06/25 10:00, Alvaro Herrera wrote:
>> On 2020-Jun-17, Fujii Masao wrote:
>>
>>> The document explains that restart_lsn column in pg_replication_slots view is:
>>>
>>> The address (LSN) of oldest WAL which still might be required by
>>> the consumer of this slot and thus won't be automatically removed
>>> during checkpoints.
>>>
>>> But the latter part is not true in v13 thanks to max_slot_wal_keep_size.
>>> I think that we need to update it as follows. Thought?
>>>
>>> The address (LSN) of oldest WAL which still might be required by
>>> the consumer of this slot and thus won't be automatically removed
>>> during checkpoints unless this LSN gets behind more than
>>> max_slot_wal_keep_size from the current LSN.
>>
>> We just added the invalidated_at LSN to replication slots; while working
>> on the tests for that today, I was thinking that it might be useful to
>> display that LSN in pg_replication_slots. What do you think of the idea
>> of publishing the invalidated_at LSN in pg_replication_slot.restart_lsn
>> when the slot is invalid?
>
> I like having separate column for invalidated_at because (at least for me)
> it's a bit confusing to report the different meaning values in the same column
> depending on the state.
Is there any other objection to the patch? If nothing, I'd like to push it.
Regards,
--
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, alvherre@2ndquadrant.com, pgsql-docs@lists.postgresql.org
Subject: Re: The description for pg_replication_slots.restart_lsn
In-Reply-To: <b495e29a-4131-81f6-7d0d-dcbb51cc5152@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