On Wed, Nov 16, 2016 at 8:25 AM, Etsuro Fujita <fujita.ets...@lab.ntt.co.jp> wrote: > On 2016/11/15 19:04, Rushabh Lathia wrote: >> >> Thanks Fujita-san for working on this. I've signed up to review this >> patch. > > > Thank you for reviewing the patch! > >> Your latest patch doesn't not get apply cleanly apply on master branch. > > > Did you apply the patch set in [1] (postgres-fdw-subquery-support-v4.patch > and postgres-fdw-phv-pushdown-v4.patch in this order) before applying the > latest patch? >
I don't see any reason why DML/UPDATE pushdown should depend upon subquery deparsing or least PHV patch. Combined together they can help in more cases, but without those patches, we will be able to push-down more stuff. Probably, we should just restrict push-down only for the cases when above patches are not needed. That makes reviews easy. Once those patches get committed, we may add more functionality depending upon the status of this patch. Does that make sense? -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
