Re: DTrace probe patch for OS X Leopard

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: DTrace probe patch for OS X Leopard
Дата
Msg-id 22697.1204263875@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: DTrace probe patch for OS X Leopard  (Robert Lor <Robert.Lor@Sun.COM>)
Ответы Re: DTrace probe patch for OS X Leopard  (Robert Lor <Robert.Lor@Sun.COM>)
Список pgsql-patches
Robert Lor <Robert.Lor@Sun.COM> 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

В списке pgsql-patches по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: Fix for initdb failures on Vista
Следующее
От: Tom Lane
Дата:
Сообщение: Re: tuplestore_putvalues()