Lists: | pgsql-hackers |
---|
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | removing =>(text, text) in 9.2 |
Date: | 2011-11-01 18:19:12 |
Message-ID: | CA+TgmoZJ_ejfVj0oCLp0uSXfcEEhnOwSOGNhqBA2+5acTtzVgw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Thu, Oct 6, 2011 at 12:46 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> +1. However, if that's the route we're traveling down, I think we had
>> better go ahead and remove the one remaining => operator from hstore
>> in 9.2:
>
> Fair enough.
So, I tried to work up a patch for this, but I'm actually a bit
confused about what needs to be done here. I'll attach what I've got
so far as a starting point for discussion.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
hstore-drop-arrow.patch | application/octet-stream | 14.0 KB |
From: | "David E(dot) Wheeler" <david(at)kineticode(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-02 04:36:58 |
Message-ID: | 8DEF891F-B5A6-4D46-9B4F-45A7C2AC1E0A@kineticode.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Nov 1, 2011, at 11:19 AM, Robert Haas wrote:
>> Fair enough.
>
> So, I tried to work up a patch for this, but I'm actually a bit
> confused about what needs to be done here. I'll attach what I've got
> so far as a starting point for discussion.
Looks reasonable, if verbose. (Yes, the extension design currently requires that). What about doc updates?
Best,
David
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | "David E(dot) Wheeler" <david(at)kineticode(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-02 14:49:42 |
Message-ID: | CA+Tgmoa8TkEwO8qNh0GLiUgNHsjFcn2e2L471e7mTr80KWgQSw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Wed, Nov 2, 2011 at 12:36 AM, David E. Wheeler <david(at)kineticode(dot)com> wrote:
> On Nov 1, 2011, at 11:19 AM, Robert Haas wrote:
>>> Fair enough.
>>
>> So, I tried to work up a patch for this, but I'm actually a bit
>> confused about what needs to be done here. I'll attach what I've got
>> so far as a starting point for discussion.
>
> Looks reasonable, if verbose. (Yes, the extension design currently requires that). What about doc updates?
Hmm, I was kind of expecting that to be wrong at least in some minor way.
Here's a new version with documentation updates.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
hstore-drop-arrow-v2.patch | application/octet-stream | 15.3 KB |
From: | Dimitri Fontaine <dimitri(at)2ndQuadrant(dot)fr> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | "David E(dot) Wheeler" <david(at)kineticode(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-03 14:18:52 |
Message-ID: | m21utpuw4z.fsf@2ndQuadrant.fr |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> Hmm, I was kind of expecting that to be wrong at least in some minor way.
> +/* contrib/hstore/hstore-1.0-1.1.sql */
> +
> +-- complain if script is sourced in psql, rather than via CREATE EXTENSION
> +\echo Use "ALTER EXTENSION hstore" to load this file. \quit
You could mention ALTER EXTENSION hstore UPDATE TO 1.1; in this comment,
I think.
> +++ b/contrib/hstore/hstore--1.1.sql
> @@ -0,0 +1,524 @@
> +/* contrib/hstore/hstore--1.0.sql */
That needs a comment update too.
Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr> |
Cc: | "David E(dot) Wheeler" <david(at)kineticode(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-04 14:28:23 |
Message-ID: | CA+TgmoazD6UaO2w=SmoNqA+oxQnfRh44Sa-4hQTp=Nw8R1uPkQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Thu, Nov 3, 2011 at 10:18 AM, Dimitri Fontaine
<dimitri(at)2ndquadrant(dot)fr> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> Hmm, I was kind of expecting that to be wrong at least in some minor way.
>
>> +/* contrib/hstore/hstore-1.0-1.1.sql */
>> +
>> +-- complain if script is sourced in psql, rather than via CREATE EXTENSION
>> +\echo Use "ALTER EXTENSION hstore" to load this file. \quit
>
> You could mention ALTER EXTENSION hstore UPDATE TO 1.1; in this comment,
> I think.
>
>> +++ b/contrib/hstore/hstore--1.1.sql
>> @@ -0,0 +1,524 @@
>> +/* contrib/hstore/hstore--1.0.sql */
>
> That needs a comment update too.
Thanks for the review. New version attached.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
hstore-drop-arrow-v3.patch | application/octet-stream | 15.3 KB |
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>, "David E(dot) Wheeler" <david(at)kineticode(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-07 19:37:20 |
Message-ID: | 688.1320694640@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> Thanks for the review. New version attached.
This looks sane to me too (though I only read the patch and didn't
actually test upgrading).
regards, tom lane
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>, "David E(dot) Wheeler" <david(at)kineticode(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: removing =>(text, text) in 9.2 |
Date: | 2011-11-08 02:55:16 |
Message-ID: | CA+Tgmoaf-4M4pdTcdnnprR3fCF=+mxvEV-HW-9QRCbvDLmWkGQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Mon, Nov 7, 2011 at 2:37 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> Thanks for the review. New version attached.
>
> This looks sane to me too (though I only read the patch and didn't
> actually test upgrading).
OK, committed.
Thanks for the reviews, all.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company