Re: INSERT ... ON CONFLICT syntax issues

From: Petr Jelinek <petr(at)2ndquadrant(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>, hlinnaka(at)iki(dot)fi, Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Bruce Momjian <bruce(at)momjian(dot)us>, Jeff Janes <jeff(dot)janes(at)gmail(dot)com>
Subject: Re: INSERT ... ON CONFLICT syntax issues
Date: 2015-04-26 11:02:52
Message-ID: 553CC5DC.9030503@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 26/04/15 12:08, Andres Freund wrote:
> On April 26, 2015 11:22:01 AM GMT+02:00, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> wrote:
>> On 04/25/2015 12:01 PM, Andres Freund wrote:
>>>
>>> That's why I wanted the WHERE outside the (), which requires either
>>> adding DO between the index inference clause, and the action, to
>> avoid
>>> ambiguities in the grammar.
>>
>> Yeah, having the WHERE outside the parens seems much nicer. What is the
>>
>> ambiguity?
>
>
> With a full keyword in between (like DO), there's none. But without it its ambiguous where a trailing UPDATE belongs to. At least from the point of a LALR grammar. WHERE UPDATE; is legal. I don't see the DO as much of a problem though.
>

The DO variant with WHERE outside of parenthesis sounds fine to me. Or
at least better than the alternatives I've seen or can come up with.

>>> A couple things:
>>>
>>> a) Why is is 'CONFLICT"? We're talking about a uniquness violation.
>> What
>>> if we, at some later point, also want to handle other kind of
>>> violations? Shouldn't it be ON UNIQUE CONFLICT/ERROR/VIOLATION
>> ...
>>
>> As Peter said, it's also for exclusion constraints. Perhaps "ON
>> CONSTRAINT VIOLATION"? It doesn't apply to foreign key constraints,
>> though. I think "ON CONFLICT" is fine.
>
> What if we, as at least I have previously wished for, want to allow handling other types of constraints? It'd be quite cool to be able to insert the referenced key on a fkey violation for some use cases.
>
>>> b) For me there's a WITH before the index inference clause missing,
>> to
>>> have it read in 'SQL' style.
>>
>> Agreed. ON would sound more natural than WITH though:
>>
>> INSERT INTO mytable ON CONFLICT ON (keycol) UPDATE ...
>
> I chose WITh because of the repeated DO; that's all ;)
>

The ON CONFLICT ON sounds really weird to me. Either ON CONSTRAINT
VIOLATION (foo) or ON CONFLICT [WITH] (foo) both seem acceptable.

--
Petr Jelinek http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabien COELHO 2015-04-26 12:30:48 Re: improve pgbench syntax error messages
Previous Message Andres Freund 2015-04-26 10:08:24 Re: INSERT ... ON CONFLICT syntax issues