Robert Lor <[EMAIL PROTECTED]> writes:
> Peter Eisentraut wrote:
>> Is c.h the right place to include this?  The probes are only in the backend 
>> code, so perhaps postgres.h would be more appropriate.  Or just include it 
>> in 
>> the .c files that need it, of which there are only 3.
>> 
> I think putting probes.h in c.h is the right place. It's true that the 
> probes are only in the backend now and only in a few files, but in the 
> future I can foresee probes added to more files in the backend, the 
> procedural language code or any of the commands (initdb, psql, etc).

I agree with Peter.  There are a whole lot of include files that are
needed by way more than 3 .c files, and yet are not folded into
postgres.h.  c.h is right out.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
       choose an index scan if your joining column's datatypes do not
       match

Reply via email to