Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Jun 20, 2019 at 4:54 AM Dilip Kumar <dilipbalaut@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
