On 2014-02-26 15:30:55 -0300, Alvaro Herrera wrote: > Andres Freund escribió: > > > I am wondering about the related situation of GetOldestXmin() > > callers. There's a fair bit of duplicated logic in the callers, before > > but especially after this patchset. What about adding 'Relation rel' > > parameter instead of `allDbs' and `systable'? That keeps the logic > > centralized and there's been a fair amount of talk about vacuum > > optimizations that could also use it. > > It's a bit sad that that requires including rel.h from procarray.h... > > relcache.h, not rel.h.
RelationData is declared in rel.h, not relcache.h, no? Alternatively we could just forward declare it in the header... Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
