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