On Thu, Mar 21, 2024 at 5:19 AM Bharath Rupireddy <bharath.rupireddyforpostg...@gmail.com> wrote: > > On Wed, Mar 20, 2024 at 7:08 PM Bertrand Drouvot > <bertranddrouvot...@gmail.com> wrote: > > > > Regarding v12-0004: "Allow setting inactive_timeout in the replication > > command", > > shouldn't we also add an new SQL API say: pg_alter_replication_slot() that > > would > > allow to change the timeout property? > > > > That would allow users to alter this property without the need to make a > > replication connection. > > +1 to add a new SQL function pg_alter_replication_slot(). >
I also don't see any obvious problem with such an API. However, this is not a good time to invent new APIs. Let's keep the feature simple and then we can extend it in the next version after more discussion and probably by that time we will get some feedback from the field as well. > > It helps > first create the slots and then later decide the appropriate > inactive_timeout. It might grow into altering other slot parameters > such as failover (I'm not sure if altering failover property on the > primary after a while makes it the right candidate for syncing on the > standby). Perhaps, we can add it for altering just inactive_timeout > for now and be done with it. > > FWIW, ALTER_REPLICATION_SLOT was added keeping in mind just the > failover property for logical slots, that's why it emits an error > "cannot use ALTER_REPLICATION_SLOT with a physical replication slot" > > > But the issue is that it would make it inconsistent with the new > > inactivetimeout > > in the subscription that is added in "v12-0005". > > Can you please elaborate what the inconsistency it causes with > inactivetimeout? > I think the inconsistency can arise from the fact that on publisher one can change the inactive_timeout for the slot corresponding to a subscription but the subscriber won't know, so it will still show the old value. If we want we can document this as a limitation and let users be aware of it. However, I feel at this stage, let's not even expose this from the subscription or maybe we can discuss it once/if we are done with other patches. Anyway, if one wants to use this feature with a subscription, she can create a slot first on the publisher with inactive_timeout value and then associate such a slot with a required subscription. -- With Regards, Amit Kapila.
