Skip to content

Commit 6331de1

Browse files
committed
Fix some obsolete information in src/backend/optimizer/README.
Constant quals aren't handled the same way they used to be. Also, add mention of a couple more major steps in grouping_planner. Per complaint a couple months back from Etsuro Fujita.
1 parent 732758d commit 6331de1

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/backend/optimizer/README

+2-5
Original file line numberDiff line numberDiff line change
@@ -297,9 +297,6 @@ planner()
297297
handle UNION/INTERSECT/EXCEPT, GROUP BY, HAVING, aggregates,
298298
ORDER BY, DISTINCT, LIMIT
299299
--query_planner()
300-
pull out constant quals, which can be used to gate execution of the
301-
whole plan (if any are found, we make a top-level Result node
302-
to do the gating)
303300
make list of base relations used in query
304301
split up the qual into restrictions (a=1) and joins (b=c)
305302
find qual clauses that enable merge and hash joins
@@ -318,11 +315,11 @@ planner()
318315
Back at standard_join_search(), apply set_cheapest() to extract the
319316
cheapest path for each newly constructed joinrel.
320317
Loop back if this wasn't the top join level.
321-
Back at query_planner:
322-
put back any constant quals by adding a Result node
323318
Back at grouping_planner:
319+
convert Path tree returned by query_planner into a Plan tree
324320
do grouping(GROUP)
325321
do aggregates
322+
do window functions
326323
make unique(DISTINCT)
327324
make sort(ORDER BY)
328325
make limit(LIMIT/OFFSET)

0 commit comments

Comments
 (0)