Re: [PATCHES] Allow the identifier length to be increased via a configure option - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCHES] Allow the identifier length to be increased via a configure option
Date
Msg-id 22139.1167251388@sss.pgh.pa.us
Whole thread Raw
Responses Re: [PATCHES] Allow the identifier length to be increased via a  (Dhanaraj M <Dhanaraj.M@Sun.COM>)
Re: [PATCHES] Allow the identifier length to be increased via a configure option  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-hackers
Alvaro Herrera <alvherre@commandprompt.com> writes:
> Dhanaraj M wrote:
>> I am sending the patch for the following TODO item:
>> Allow the identifier length to be increased via a configure option

> You should use pg_config.h, not mangle postgres_ext.h like that.  Or
> maybe generate postgres_ext.h from an hypotetical postgres_ext.h.in (but
> I wouldn't do that, really).

I'm wondering how this got into the TODO list.  It seems rather
pointless, and likely to create client compatibility problems (if not,
why is NAMEDATALEN exported at all?)

A bigger problem with the proposed patch is that it makes it look like
you can set the identifier length to any random number; the restrictions
mentioned in postgres_ext.h aren't enforced or even displayed to the
user of configure.

            regards, tom lane

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: psql display of Unicode combining characters in 8.2
Next
From: mark@mark.mielke.cc
Date:
Subject: Re: Bitmap index thoughts