Magnus Hagander <> writes:
> On Mon, Aug 15, 2011 at 16:20, Tom Lane <> wrote:
>> However, for a narrow fix, I could see moving the data type definition
>> to someplace with fewer dependencies.  Perhaps split it into a separate
>> file timestamp_type.h, or something like that.

> Yes, that seems to fix the problem of timestamptz. See the attached
> patch - seems ok?

Don't think you should expose fsec_t, nor most of those macros.  The
foo_per_bar values are just namespace clutter.

> I also ran into a similar problem with some WAL macro definitions that
> are in xlog_internal.h. I've moved them to xlogdefs.h in the attached
> xlog.diff file. Does that seem ok as well, or should I move them
> somewhere else?

I don't like the idea of exposing those to frontends, either.  What do
you actually *need* out of that, and why?

                        regards, tom lane

Sent via pgsql-hackers mailing list (
To make changes to your subscription:

Reply via email to