On 15/03/2016 14:18, Stephen Frost wrote:
> * Julien Rouhaud (julien.rouh...@dalibo.com) wrote:
>> On 15/03/2016 03:30, Peter Geoghegan wrote:
>>> On Mon, Mar 7, 2016 at 1:46 PM, Peter Geoghegan <p...@heroku.com> wrote:
>>>> Attached patch fixes a bug reported privately by Stephen this morning.
>>> I would like to see this in the next point release. It shouldn't be
>>> hard to review.
>> + reason_wait = indexInfo->ii_ExclusionOps ?
>> + XLTW_RecheckExclusionConstr : XLTW_InsertIndex;
>> Shouldn't it be set to XLTW_InsertIndexUnique instead?
> Actually, no, though I had the same question for Peter when I was first
> reviewing this.
> XLTW_InsertIndexUnique is used when building a unique index, but this is
> just a check, and more to the point, it's actually a re-check of what
> we're doing in nbinsert.c where we're already using XLTW_InsertIndex.
> We wouldn't want to end up returning different error messages for the
> same command under the same conditions just based, which is what we'd
> potentially end up doing if we used XLTW_InsertIndexUnique here.
Oh I see. Thanks for the explanation!
>> Otherwise the patch seems ok to me.
> Agreed. I'm going to play with it a bit more but barring objections,
> I'll commit and back-patch Peter's patch.
http://dalibo.com - http://dalibo.org
Sent via pgsql-hackers mailing list (firstname.lastname@example.org)
To make changes to your subscription: