Hi, On 2021-12-29 11:34:53 -0800, SATYANARAYANA NARLAPURAM wrote: > On Wed, Dec 29, 2021 at 11:31 AM Andres Freund <and...@anarazel.de> wrote: > Andres, thanks for the comments. Agreed on this based on the previous > discussions on this thread. Could you please share your thoughts on adding > it after SyncRepWaitForLSN()?
I don't think that's good either - you're delaying transaction commit (i.e. xact becoming visible / locks being released). That also has the danger of increasing lock contention (albeit more likely to be heavyweight locks / serializable state). It'd have to be after the transaction actually committed. Greetings, Andres Freund
