Re: array type name mangling

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

In response to

Responses

Browse pgsql-hackers by date

  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