On 2021-Jan-25, Andres Freund wrote: > See attached patch...
Looks good to me. I was wondering if there would be a point in using a FullTransactionId instead of an unadorned one. I don't know what's the true risk of an Xid wraparound occurring here, but it seems easier to reason about. But then that's probably a larger change to make all of snapbuild use FullTransactionIds, so not for this patch to worry about. -- Álvaro Herrera 39°49'30"S 73°17'W