On Fri, Mar 18, 2016 at 10:32 PM, David Rowley <david.row...@2ndquadrant.com> wrote: > Updated patch is attached.
I think this looks structurally correct now, and I think it's doing the right thing as far as parallelism is concerned. I don't see any obvious problems in the rest of it, either, but I haven't thought hugely deeply about the way you are doing the costing, nor have I totally convinced myself that all of the PathTarget and setrefs stuff is correct. But I think it's probably pretty close. I'll study it some more next week. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers