From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Patches (PostgreSQL)" <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: remove convert using |
Date: | 2007-09-24 01:30:33 |
Message-ID: | 46F71339.9010401@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Tom Lane wrote:
>>
>>> You should be able to remove CONVERT as a grammar keyword altogether
>>> -- the remaining production for CONVERT as a function name seems dead
>>> weight now (not to mention that it prevents having user-defined
>>> functions named CONVERT).
>>>
>
>
>> I wonderted a bit about that. I thought it might be better to leave it
>> in case we wanted to put back "convert using" when we have better
>> support for multiple encodings (and maybe when we understand better what
>> it is actually supposed to do).
>>
>
> Well, we could always put it back when we need it --- in the meantime,
> every extra keyword is some fractional drag on parsing performance.
>
> In any case I think the remaining production is probably wrong because
> it constrains the function to be in pg_catalog schema, when there is
> no grammatical evidence that it should be special.
>
OK, fix committed doing this.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2007-09-24 05:28:21 | Re: curious regression failures |
Previous Message | Tom Lane | 2007-09-24 00:27:30 | Re: remove convert using |