On Fri, Apr 17, 2020 at 3:44 AM Mark Dilger <mark.dil...@enterprisedb.com> wrote: > Hmm, I should have spoken sooner... > > src/backend/replication/walsender.c:static bool > TransactionIdInRecentPast(TransactionId xid, uint32 epoch); > src/backend/utils/adt/xid8funcs.c:TransactionIdInRecentPast(FullTransactionId > fxid, TransactionId *extracted_xid) > > I don't care much for having two different functions with the same name and > related semantics but different argument types.
Maybe that's not ideal, but it's not because of this patch. Those functions are from 5737c12df05 and 857ee8e391f.