Yeah, I agree. Sorry for missing that. The updated patch looks good to me.
On Mon, Sep 6, 2021 at 3:25 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > On Tue, Aug 17, 2021 at 11:58 AM Ashutosh Bapat > <ashutosh.ba...@enterprisedb.com> wrote: > > > > Hi Amit and Andres, > > Here's updated patch > > > > I think we can log the required information immediately after > releasing spinlock, otherwise, the other logs from > LogicalConfirmReceivedLocation() might interleave. I have made that > change and slightly edit the comment and commit message. I am planning > to push this tomorrow unless you or others have any comments. > > -- > With Regards, > Amit Kapila. > -- -- Best Wishes, Ashutosh
