On Tue, May 26, 2026 at 3:50 PM SATYANARAYANA NARLAPURAM <[email protected]> wrote: > > HI hackers, > > When ALTER SUBSCRIPTION SET (retain_dead_tuples = true, origin = 'none') > is used, the publisher version/recovery check was bypassed because the > origin handling unconditionally overwrote check_pub_rdt to false. > > Fix by using |= instead of = so that the flag set by retain_dead_tuples > cannot be cleared by a subsequent origin assignment in the same command. >
Thanks for catching this. Your fix LGTM. I won't be able to commit today as I have to travel. In the meantime, let's see if Hou-San or others have any comments. -- With Regards, Amit Kapila.
