Andres Freund escribió:
> 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?
Sure, but with your patch AFAICT procarray.h header only needs Relation,
which is declared in relcache.h.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services