On Thu, Sep 17, 2015 at 9:33 AM, Tom Lane <[email protected]> wrote: > Robert Haas <[email protected]> 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 ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
