On Sun, May 24, 2015 at 5:16 PM, Peter Geoghegan <p...@heroku.com> wrote:
> AddForeignUpdateTargets() actually won't be called with ON CONFLICT DO
> UPDATE, and so it isn't exactly true that the only obstacle to making
> FDWs support ON CONFLICT DO UPDATE is around inference of arbiter
> unique indexes on the foreign side. It's *almost* true, though.

Attached patch clears this up within the fdw-handler documentation. I
think it's worth separately noting from the existing commentary on
limitations with FDWs and ON CONFLICT.

-- 
Peter Geoghegan
diff --git a/doc/src/sgml/fdwhandler.sgml b/doc/src/sgml/fdwhandler.sgml
index 2361577..569a22d 100644
--- a/doc/src/sgml/fdwhandler.sgml
+++ b/doc/src/sgml/fdwhandler.sgml
@@ -399,6 +399,13 @@ AddForeignUpdateTargets (Query *parsetree,
     </para>
 
     <para>
+     Note that <function>AddForeignUpdateTargets</> will not be called
+     for <command>INSERT</> operations with an <literal>ON CONFLICT DO
+     UPDATE</> clause.  Such <command>INSERT</> operations are
+     unsupported when a foreign table is targeted.
+    </para>
+
+    <para>
 <programlisting>
 List *
 PlanForeignModify (PlannerInfo *root,
-- 
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