On 10 February 2016 at 08:00, Rushabh Lathia <rushabh.lat...@gmail.com> wrote: > Fujita-san, I am attaching update version of the patch, which added > the documentation update. > > Once we finalize this, I feel good with the patch and think that we > could mark this as ready for committer.
I find this wording a bit confusing: "If the IsForeignRelUpdatable pointer is set to NULL, foreign tables are assumed to be insertable, updatable, or deletable either the FDW provides ExecForeignInsert,ExecForeignUpdate, or ExecForeignDelete respectively or if the FDW optimizes a foreign table update on a foreign table using PlanDMLPushdown (PlanDMLPushdown still needs to provide BeginDMLPushdown, IterateDMLPushdown and EndDMLPushdown to execute the optimized update.)." This is a very long sentence, and the word "either" doesn't work here. Also: "When the query doesn't has the clause, the FDW must also increment the row count for the ForeignScanState node in the EXPLAIN ANALYZE case." Should read "doesn't have" The rest looks fine AFAICT. Regards Thom -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers