On Thu, Feb 6, 2025 at 10:17 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Thu, Feb 6, 2025 at 8:02 AM Nisha Moond <nisha.moond...@gmail.com> wrote: > > > > > > > > 2. > > > + * Flush all replication slots to disk. Also, invalidate obsolete slots > > > during > > > + * non-shutdown checkpoint. > > > * > > > * It is convenient to flush dirty replication slots at the time of > > > checkpoint. > > > * Additionally, in case of a shutdown checkpoint, we also identify the > > > slots > > > @@ -1924,6 +2007,45 @@ CheckPointReplicationSlots(bool is_shutdown) > > > > > > Can we try and see how the patch looks if we try to invalidate the > > > slot due to idle time at the same time when we are trying to > > > invalidate due to WAL? > > > > > > > I'll consider the suggested change in the next version. > > > > FYI, we discussed this previously (1), but the conclusion that it > won't help much (as it will not help to remove WAL immediately) is > incorrect, especially if we do what is suggested now. >
The above sentence is incomplete. Let me re-write it. We discussed this previously, but the conclusion that it won't help much (as it will not help to remove WAL immediately) at the time shutdown checkpoint is incorrect, especially if we do what is suggested now. So, we should try to invalidate the slots even during shutdown checkpoints. -- With Regards, Amit Kapila.
