Re: [PATCHES] [pgsql-patches] Phantom Command IDs,updated patch - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCHES] [pgsql-patches] Phantom Command IDs,updated patch
Date
Msg-id 13565.1170990926@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCHES] [pgsql-patches] Phantom Command IDs,updated patch  (Alvaro Herrera <alvherre@commandprompt.com>)
Responses Re: [PATCHES] [pgsql-patches] Phantom CommandIDs,updated patch  ("Simon Riggs" <simon@2ndquadrant.com>)
Re: [PATCHES] [pgsql-patches] Phantom Command IDs,updated patch  (Heikki Linnakangas <heikki@enterprisedb.com>)
List pgsql-hackers
Alvaro Herrera <alvherre@commandprompt.com> writes:
> Simon Riggs wrote:
>> Combo is OK, because it's a *combination* of two CommandIds. 
>> 
>> That means they are ComboCommandIds or CCIs.

> CCI is CommandCounterIncrement to me, so let's not use that
> abbreviation.

Agreed.  I looked for a bit at adding a separate typedef ComboCommandId
to be used where a cid was definitely a combo cid, but it didn't seem to
be very useful to do that...

I'm testing the patch currently.  I was a bit surprised to find the
without_oid test failing, but it makes sense because I'm using a
MAXALIGN=8 machine.  I suppose Heikki tested on MAXALIGN=4.
        regards, tom lane


pgsql-hackers by date:

Previous
From: "Simon Riggs"
Date:
Subject: Re: [PATCHES] [pgsql-patches] Phantom CommandIDs,updated patch
Next
From: "Joshua D. Drake"
Date:
Subject: Re: Proposal: Commit timestamp