On Wed, 20 Sept 2023 at 16:54, Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Fri, Sep 15, 2023 at 3:08 PM vignesh C <vignes...@gmail.com> wrote: > > > > The attached v8 version patch has the changes for the same. > > > > Is the check to ensure remote_lsn is valid correct in function > check_for_subscription_state()? How about the case where the apply > worker didn't receive any change but just marked the relation as > 'ready'?
I agree that remote_lsn will not be valid in the case when all the tables are in ready state and there are no changes to be sent by the walsender to the worker. I was not sure if this check is required in this case in the check_for_subscription_state function. I was thinking that this check could be removed. I'm also checking why the tables should only be in ready state, the check that is there in the same function, can we support upgrades when the tables are in syncdone state or not. I will post my analysis once I have finished checking on the same. Regards, Vignesh
