Re: shared-memory based stats collector

On 11/9/18 9:33 AM, Kyotaro HORIGUCHI wrote:
> Hello. This is rebased version.
> 
> At Thu, 8 Nov 2018 16:06:49 +0100, Tomas Vondra <tomas.vondra@2ndquadrant.com> wrote in
<de249c3f-79c9-b75c-79a3-5e2d008548a8@2ndquadrant.com>
>> However a quite a few extensions in contrib seem are broken now. It
>> seems fixing it is as simple as including the new bestatus.h next to
>> pgstat.h.
> 
> The additional 0009 does that.
> 

That does fix it, indeed. But the break happens in 0003, so that's where 
the fixes should be moved - I've tried to simply apply 0009 right after 
0003, but that does not seem to work because bestatus.h does not exist 
at that point yet :-/

The current split into 8 parts seems quite sensible to me, i.e. that's 
how it might get committed eventually. That however means each part 
needs to be correct on it's own (hence fixes in 0009 are a problem).

regards

-- 
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services