On Thursday, June 28, 2012 10:03:26 PM Andres Freund wrote:
> What I wonder is how hard it would be to remove catcache.h's structs into
> the implementation. Thats the reason why the old and new list
> implementation currently is included all over the backend...
Moving them into the implementation isn't possible, but catcache.h being
included just about everywhere simply isn't needed.
It being included everywhere was introduced by a series of commits from Bruce:
That looks broken. An implementation file not including its own header... A
minimal patch to fix this particular problem is attached (looks like there are
others in the series).
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
In response to
pgsql-hackers by date
|Next:||From: Alvaro Herrera||Date: 2012-06-28 21:09:03|
|Subject: Re: embedded list v2|
|Previous:||From: Peter Geoghegan||Date: 2012-06-28 20:32:40|
|Subject: Re: Uh, I change my mind about commit_delay +
commit_siblings (sort of)|