Robert Haas <robertmh...@gmail.com> writes: > On Thu, Jun 20, 2019 at 4:54 AM Dilip Kumar <dilipbal...@gmail.com> wrote: >> The idea behind having the loop inside the undo machinery was that >> while traversing the blkprev chain, we can read all the undo records >> on the same undo page under one buffer lock.
> That's not a bad goal, although invoking a user-supplied callback > while holding a buffer lock is a little scary. I nominate Robert for Understater of the Year. I think there's pretty much 0 chance of that working reliably. regards, tom lane