Re: hstore ==> and deprecate => - Mailing list pgsql-hackers

From Robert Haas
Subject Re: hstore ==> and deprecate =>
Date
Msg-id AANLkTil1bkv-VFS1x8JegX9hHxBbm3GM8_YOiTUlccxU@mail.gmail.com
Whole thread Raw
In response to Re: hstore ==> and deprecate =>  (Greg Stark <gsstark@mit.edu>)
List pgsql-hackers
On Wed, Jun 9, 2010 at 6:41 AM, Greg Stark <gsstark@mit.edu> wrote:
> On Tue, Jun 8, 2010 at 8:07 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>> I believe that the consensus was mostly in favor of deprecating => as
>> an operator name, with the intent to abolish it completely in a future
>> release.  Attached is a patch to implement ==> as an alternative
>> operator name for hstore, and to make the backend throw a warning when
>> => is used as an operator name.
>
> I don't think we can throw warnings for DML except in the most dire
> circumstances.

The patch doesn't.

>> One wart is that => is used not only as a SQL-level operator, but also
>> by hstore_in() when interpreting hstore-type literals, and by
>> hstore_out() when generating them.  My gut feeling is that we should
>> leave this part alone and only muck with the SQL operator, but perhaps
>> someone will care to argue the point.
>
> I hate these kinds of inconsistencies. I would prefer both operators
> be consistent.

Sure, me too, but I think this is the best we can do.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


pgsql-hackers by date:

Previous
From: Greg Stark
Date:
Subject: Re: hstore ==> and deprecate =>
Next
From: Dimitri Fontaine
Date:
Subject: Re: hstore ==> and deprecate =>