On Mon, Sep 18, 2017 at 7:02 AM, Rushabh Lathia <rushabh.lat...@gmail.com> wrote: >> In that case, can you please mark the patch [1] as ready for committer in >> CF app > > Done.
I think this patch is mostly correct, but I think the change to planner.c isn't quite right. ordered_rel->reltarget is just a dummy target list that produces nothing. Instead, I think we should pass path->pathtarget, representing the idea that whatever Gather Merge produces as output is the same as what you put into it. See attached. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
pushdown-gathermerge-tlist.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers