On Monday, June 26, 2023 12:18 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Fri, Jun 23, 2023 at 10:07 PM Andres Freund <and...@anarazel.de> wrote: > > > > On 2023-06-23 14:04:15 +0530, Amit Kapila wrote: > > > OTOH, if the above theory is wrong or people are not convinced, I am > > > okay with removing all the changes in commits 72e78d831a and > > > 3ba59ccc89. > > > > I am not convinced. And even if I were, coming up with new > > justifications in a released version, when the existing testing > > clearly wasn't enough to find the current bug, doesn't strike me as wise. > > > > Fair enough. If we could have been convinced of this then we can keep the > required change only for HEAD. But anyway let's remove the work related to > both commits (72e78d831a and 3ba59ccc89) for now and then we can come > back to it when we parallelize writes. The attached patch removes the changes > added by both commits with slight tweaking in comments/readme based on > the recent state.
Thanks for the patch. I have confirmed that the patch to revert page lock handling applies cleanly on all branches(13~HEAD) and the assert failure and undetectable deadlock problem are fixed after applying the patch. Best Regards, Hou zj
