From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: should we add a XLogRecPtr/LSN SQL type? |
Date: | 2014-02-24 14:47:01 |
Message-ID: | CA+TgmoatbE1U3GKD=KXC_rvVFucTowW0fXuu+BA0ZNdjzYNHMw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Feb 19, 2014 at 8:06 PM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> On Thu, Feb 20, 2014 at 9:43 AM, Michael Paquier
> <michael(dot)paquier(at)gmail(dot)com> wrote:
>> On Thu, Feb 20, 2014 at 2:47 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> On Wed, Feb 5, 2014 at 9:26 PM, Michael Paquier
>>> <michael(dot)paquier(at)gmail(dot)com> wrote:
>>>> On Thu, Feb 6, 2014 at 3:48 AM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
>>>>> On 2/5/14, 1:31 PM, Robert Haas wrote:
>>>>>> On Tue, Feb 4, 2014 at 3:26 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
>>>>>>> Perhaps this type should be called pglsn, since it's an
>>>>>>> implementation-specific detail and not a universal concept like int,
>>>>>>> point, or uuid.
>>>>>>
>>>>>> If we're going to do that, I suggest pg_lsn rather than pglsn. We
>>>>>> already have pg_node_tree, so using underscores for separation would
>>>>>> be more consistent.
>>>>>
>>>>> Yes, that's a good precedent in multiple ways.
>>>> Here are updated patches to use pg_lsn instead of pglsn...
>>>
>>> OK, so I think this stuff is all committed now, with assorted changes.
>>> Thanks for your work on this.
>> Thanks!
>> Oops, it looks like I am coming after the battle (time difference does
>> not help). I'll be more careful to test such patches on 32b platforms
>> as well in the future.
> After re-reading the code, I found two incorrect comments in the new
> regression tests. Patch fixing them is attached.
Thanks, committed. But I left out the whitespace change you included.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2014-02-24 14:48:26 | Re: Changeset Extraction v7.6.1 |
Previous Message | Merlin Moncure | 2014-02-24 14:46:06 | Re: jsonb and nested hstore |