diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-03-08 16:28:27 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-03-08 16:28:34 -0500 |
commit | 8c314b9853c2fbb85c041d4761426f25a9d63972 (patch) | |
tree | 4f8f5c1ac067589c112c71f25331e79ae0147e56 /contrib/postgres_fdw/postgres_fdw.c | |
parent | 7400559a3fe959bdc6822114ef6aa966c42899f9 (diff) | |
download | postgresql-8c314b9853c2fbb85c041d4761426f25a9d63972.tar.gz postgresql-8c314b9853c2fbb85c041d4761426f25a9d63972.zip |
Finish refactoring make_foo() functions in createplan.c.
This patch removes some redundant cost calculations that I left for later
cleanup in commit 3fc6e2d7f5b652b4. There's now a uniform policy that the
make_foo() convenience functions don't do any cost calculations. Most of
their callers copy costs from the source Path node, and for those that
don't, the calculation in the make_foo() function wasn't necessarily right
anyhow. (make_result() was particularly a mess, as it was serving multiple
callers using cost calcs designed for only the first one or two that had
ever existed.) Aside from saving a few cycles, this ensures that what
EXPLAIN prints matches the costs we used for planning purposes. It does
not change any planner decisions, since the decisions are already made.
Diffstat (limited to 'contrib/postgres_fdw/postgres_fdw.c')
0 files changed, 0 insertions, 0 deletions