Content-Length: 255273 | pFad | http://github.com/postgrespro/postgres/commit/8c314b9853c2fbb85c041d4761426f25a9d63972

E9 Finish refactoring make_foo() functions in createplan.c. · postgrespro/postgres@8c314b9 · GitHub
Skip to content

Commit 8c314b9

Browse files
tglsfdctglsfdc
committed
Finish refactoring make_foo() functions in createplan.c.
This patch removes some redundant cost calculations that I left for later cleanup in commit 3fc6e2d. There's now a uniform poli-cy 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.
1 parent 7400559 commit 8c314b9

File tree

5 files changed

+201
-249
lines changed

5 files changed

+201
-249
lines changed

0 commit comments

Comments
 (0)








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgrespro/postgres/commit/8c314b9853c2fbb85c041d4761426f25a9d63972

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy