On Tue, Apr 5, 2016 at 12:52 PM, Petr Jelinek <petr@2ndquadrant.com> wrote: > >> I have just one question about de 0002 patch: >> - There are some reason to not use TransactionId instead of uint32 in >> GaplessSequenceState struct? >> > > I missed we have that :) >
Attached fix it and also I changed to use TransactionIdEquals instead of compare xids directly.
- if (seqstate->xid != local_xid) + if (!TransactionIdEquals(seqstate->xid, local_xid))
IMHO this patch is Ready for Commiter. Changed status in CommitfestApp.