From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: array type name mangling |
Date: | 2007-05-05 15:24:26 |
Message-ID: | 463CA1AA.5040909@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> In connection with completing David Fetter's array of composites patch,
>> I am looking at doing some better name mangling for array types as
>> recently discussed. What I'm thinking of is prepending one or more
>> underscores to the type name up to some limit (NAMEDATALEN / 2 ?) and if
>> necessary truncating the result, and then looking to see if there is a
>> name clash. That would, I hope, enable us to get rid of all the places
>> where we require names to be no more than NAMEDATALEN - 2 chars. Does
>> that seem like a reasonable approach? Will it break anything, i.e., is
>> there somewhere that has assumes the array type for foo will be called
>> _foo rather than ___foo ?
>>
>
> makeArrayTypeName and users thereof. Or are you going to extend pg_type
> to have a direct link?
>
>
>
I am going to change makeArrayTypeName() to do the mangling. Its users
will need to pass in a namespace as well as a typename so it can do the
checking.
There's a direct link via typelem - do you think we need a reverse mapping?
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Neil Conway | 2007-05-05 15:38:07 | Re: Integer datetimes |
Previous Message | Tom Lane | 2007-05-05 15:21:19 | Re: Cache plan invalidation |