Re: DTrace probe patch for OS X Leopard

From: Robert Lor <Robert(dot)Lor(at)Sun(dot)COM>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: DTrace probe patch for OS X Leopard
Date: 2008-02-28 23:52:27
Message-ID: 47C7493B.4030606@sun.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

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).

Regards,
-Robert

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Neil Conway 2008-02-29 00:37:53 tuplestore_putvalues()
Previous Message Peter Eisentraut 2008-02-28 23:18:14 Re: DTrace probe patch for OS X Leopard