On Thu, Oct 28, 2021 at 8:15 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Wed, Oct 27, 2021 at 4:39 PM Dilip Kumar <dilipbal...@gmail.com> wrote: > > > > On Tue, Oct 26, 2021 at 9:19 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > > > > > > > > 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. > > > > Right, even I feel we should just commit it to the HEAD as there is no > > correctness issue. > > > > Thanks for your opinion. I'll commit it to the HEAD by next Tuesday > unless someone feels that we should backpatch this. >
Pushed. -- With Regards, Amit Kapila.
