On Wed, Aug 25, 2021 at 1:37 AM Robert Haas <robertmh...@gmail.com> wrote: > > I guess I was thinking more of rejiggering things so that we save the > results of each RestoreSnapshot() call in a local variable, e.g. > asnapshot and tsnapshot. And then I think we could just > RestoreTransactionSnapshot() on whichever one we want, and then > PushActiveSnapshot(asnapshot) either way. I think it would be worth > trying to move the PushActiveSnapshot() call out of the if statement > instead it in two places, written differently but doing the same > thing. >
I've attached an updated patch, hopefully more along the lines that you were thinking of. Regards, Greg Nancarrow Fujitsu Australia
v11-0001-Fix-parallel-worker-failed-assertion-and-coredump.patch
Description: Binary data
