On Mon, Oct 30, 2017 at 9:00 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: > Now that the PARAM_EXTERN issue is fixed, I have rebased this patch. > This patch had been switched to Ready For Committer in last CF, then > Robert had comments which I have addressed, so I think the status > should be switched back to Ready For committer. Let me know if you > think it should be switched to some other status.
The change to ExplainPrintPlan doesn't look good to me, because it actually moves the initPlan; I don't think it's good for EXPLAIN to mutate the plan state tree. It should find a way to display the results *as if* the initPlans were attached to the subnode, but without actually moving them. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers