Re: remove convert using

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

In response to

Browse pgsql-patches by date

  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