- Notifications
You must be signed in to change notification settings - Fork5
Commit8c314b9
committed
Finish refactoring make_foo() functions in createplan.c.
This patch removes some redundant cost calculations that I left for latercleanup in commit3fc6e2d. There's now a uniform policy that themake_foo() convenience functions don't do any cost calculations. Most oftheir callers copy costs from the source Path node, and for those thatdon't, the calculation in the make_foo() function wasn't necessarily rightanyhow. (make_result() was particularly a mess, as it was serving multiplecallers using cost calcs designed for only the first one or two that hadever existed.) Aside from saving a few cycles, this ensures that whatEXPLAIN prints matches the costs we used for planning purposes. It doesnot change any planner decisions, since the decisions are already made.1 parent7400559 commit8c314b9
File tree
5 files changed
+201
-249
lines changed- src
- backend/optimizer
- plan
- util
- include/optimizer
5 files changed
+201
-249
lines changed0 commit comments
Comments
(0)