On 2014-07-18 11:14:34 -0700, Peter Geoghegan wrote:
> On Fri, Jul 18, 2014 at 11:06 AM, Andres Freund <and...@2ndquadrant.com> 
> wrote:
> > I don't see why you'd need such a node at all if we had a fully builtin
> > UPSERT. The whole stuff with ON CONFLICT SELECT FOR UPDATE and then
> > UPDATE ... FROM c CONFLICTS is too complicated and exposes stuff that
> > barely anybody will understand, let alone use correctly in queries they
> > write themselves.
> 
> I accept that there will be a need for certain restrictions. Most
> obviously, if you update the target table referencing a CTE like this,
> not using the special CONFLICTS clause in the UPDATE (or DELETE) is an
> error. And as I mentioned, you may only join the projected duplicates
> to the UPDATE ModifyTable - an attempt to join any more relations is
> an error. In short, this *is* a fully built-in upsert.

Meh. A understandable syntax wouldn't require the pullups with a special
scan node and such. I think you're attempting a sort of genericity
that's making your (important!) goal much harder to reach.

Greetings,

Andres Freund

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


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to