On Mon, Oct 25, 2021 at 4:21 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Thu, Oct 21, 2021 at 11:20 AM Dilip Kumar <dilipbal...@gmail.com> wrote: > > > > On Thu, Oct 21, 2021 at 9:11 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > > > > v5-0001, incorporates all the comment fixes suggested by Alvaro. and > > 0001 is an additional patch which moves > > MarkCurrentTransactionIdLoggedIfAny(), out of the critical section. > > > > Thanks, both your patches look good to me except that we need to > remove the sentence related to the revert of ade24dab97 from the > commit message. I think we should backpatch the first patch to 14 > where it was introduced and commit the second patch (related to moving > code out of critical section) only to HEAD but we can even backpatch > the second one till 9.6 for the sake of consistency. What do you guys > think? >
The other option could be to just commit both these patches in HEAD as there is no correctness issue here. -- With Regards, Amit Kapila.
