On Tue, Jun 2, 2026 at 7:44 AM Zhijie Hou (Fujitsu) <[email protected]> wrote: > > On Saturday, May 30, 2026 6:19 AM Amit Kapila <[email protected]> wrote: > > > > 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 reporting ! I agree this needs to be fixed. > > > > > > > > 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. > > The fix looks good to me as well. >
Thanks for the verification. I've pushed the fix. -- With Regards, Amit Kapila.
