On Sun, Feb 03, 2019 at 10:58:02PM +1100, Thomas Munro wrote: > If there are no objections, I'm planning to do a round of testing and > commit this shortly.
Hm. That looks sane to me at quick glance. I am a bit on the edge regaring the naming "FullTransactionId", which is actually a 64-bit value with a 32-bit XID and a 32-bit epoch. Something like TransactionIdWithEpoch or EpochTransactionId sounds a bit better to me. My point is that "Full" is too generic for that. Moved to next CF for now. -- Michael
signature.asc
Description: PGP signature
