On Thu, Sep 17, 2015 at 9:33 AM, Tom Lane <t...@sss.pgh.pa.us> wrote: > Robert Haas <robertmh...@gmail.com> writes: >> Here is a patch that *just* introduces planstate_tree_walker and which >> is hopefully correct. I stole the logic from ExplainPreScanNode, >> which I also refactored to use the new walker instead of duplicating >> the logic. > > It seems a little odd to have removed functions from explain.c altogether > but left their header comments behind. Otherwise this seems sound.
D'oh! OK, will fix that before committing. Thanks. -- 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
