Hi,
On Wed, Sep 30, 2026 at 09:08:41AM +0530, shveta malik wrote:
> On Tue, Sep 29, 2026 at 1:59 PM Bertrand Drouvot
> <bertranddrouvot.pg@gmail.com> wrote:
> >
> > The static flag looks safe, but I wonder if it wouldn't be clearer to keep
> > ReplicationSlotReleaseInternal() and call it with false from the error path?
> >
> > That would still allow us to remove update_inactive_since from
> > ReplicationSlotPersistInvalidation() and its callers, while keeping the
> > exceptional release behavior explicit.
> >
>
> Okay, works for me.
Thanks, done that way in the attached.
Regards,
--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com