On 2 September 2016 at 23:29, Petr Jelinek <petr@2ndquadrant.com> wrote:
> You could put it to txid.c where all the other txid stuff is in?
Yeah, even though it's in adt/ I think it'll do.
I thought I'd need get_xid_in_recent_past() for catalog_xmin hot
standby feedback, but upon closer examination the needed logic isn't
the same anymore. txid_status() wants to ensure clog lookups are safe
and limit by oldest xid, wheras the walsender doesn't actually care
about that and is just avoiding wrapped xids.
I'm just going back to how it was, all in adt/txid.c, and making it
static again. We can move it and make it non-static if a need to do so
comes up.
Attached rebased patch updated and vs master.
--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services