database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-02 11:44:50 +03:00
Code

Update some comments to clarify who does what in targetlist creation.

Browse Source 
No code changes; just avoid blaming query_planner for things it doesn't
really do.
This commit is contained in:
Tom Lane 2011-07-13 20:23:09 -04:00
parent 0527a454ec
commit 96f990e23b
1 changed files with 17 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/backend/optimizer/plan/planner.c
View File

@ -1248,17 +1248,17 @@ grouping_planner(PlannerInfo *root, double tuple_fraction)
need_sort_for_grouping = true; need_sort_for_grouping = true;
/* /*
* Always override query_planner's tlist, so that we don't * Always override create_plan's tlist, so that we don't
* sort useless data from a "physical" tlist. * sort useless data from a "physical" tlist.
*/ */
need_tlist_eval = true; need_tlist_eval = true;
} }
/* /*
* create_plan() returns a plan with just a "flat" tlist of * create_plan returns a plan with just a "flat" tlist of
* required Vars. Usually we need to insert the sub_tlist as the * required Vars. Usually we need to insert the sub_tlist as the
* tlist of the top plan node. However, we can skip that if we * tlist of the top plan node. However, we can skip that if we
* determined that whatever query_planner chose to return will be * determined that whatever create_plan chose to return will be
* good enough. * good enough.
*/ */
if (need_tlist_eval) if (need_tlist_eval)
@ -1311,7 +1311,7 @@ grouping_planner(PlannerInfo *root, double tuple_fraction)
else else
{ {
/* /*
* Since we're using query_planner's tlist and not the one * Since we're using create_plan's tlist and not the one
* make_subplanTargetList calculated, we have to refigure any * make_subplanTargetList calculated, we have to refigure any
* grouping-column indexes make_subplanTargetList computed. * grouping-column indexes make_subplanTargetList computed.
*/ */
@ -2521,10 +2521,11 @@ choose_hashed_distinct(PlannerInfo *root,
* make_subplanTargetList * make_subplanTargetList
* Generate appropriate target list when grouping is required. * Generate appropriate target list when grouping is required.
* *
* When grouping_planner inserts Aggregate, Group, or Result plan nodes * When grouping_planner inserts grouping or aggregation plan nodes
* above the result of query_planner, we typically want to pass a different * above the scan/join plan constructed by query_planner+create_plan,
* target list to query_planner than the outer plan nodes should have. * we typically want the scan/join plan to emit a different target list
* This routine generates the correct target list for the subplan. * than the outer plan nodes should have. This routine generates the
* correct target list for the scan/join subplan.
* *
* The initial target list passed from the parser already contains entries * The initial target list passed from the parser already contains entries
* for all ORDER BY and GROUP BY expressions, but it will not have entries * for all ORDER BY and GROUP BY expressions, but it will not have entries
@ -2547,15 +2548,18 @@ choose_hashed_distinct(PlannerInfo *root,
* If we are grouping or aggregating, *and* there are no non-Var grouping * If we are grouping or aggregating, *and* there are no non-Var grouping
* expressions, then the returned tlist is effectively dummy; we do not * expressions, then the returned tlist is effectively dummy; we do not
* need to force it to be evaluated, because all the Vars it contains * need to force it to be evaluated, because all the Vars it contains
* should be present in the output of query_planner anyway. * should be present in the "flat" tlist generated by create_plan, though
* possibly in a different order. In that case we'll use create_plan's tlist,
* and the tlist made here is only needed as input to query_planner to tell
* it which Vars are needed in the output of the scan/join plan.
* *
* 'tlist' is the query's target list. * 'tlist' is the query's target list.
* 'groupColIdx' receives an array of column numbers for the GROUP BY * 'groupColIdx' receives an array of column numbers for the GROUP BY
* expressions (if there are any) in the subplan's target list. * expressions (if there are any) in the returned target list.
* 'need_tlist_eval' is set true if we really need to evaluate the * 'need_tlist_eval' is set true if we really need to evaluate the
* result tlist. * returned tlist as-is.
* *
* The result is the targetlist to be passed to the subplan. * The result is the targetlist to be passed to query_planner.
*/ */
static List * static List *
make_subplanTargetList(PlannerInfo *root, make_subplanTargetList(PlannerInfo *root,
@ -2649,7 +2653,7 @@ make_subplanTargetList(PlannerInfo *root,
/* /*
* locate_grouping_columns * locate_grouping_columns
* Locate grouping columns in the tlist chosen by query_planner. * Locate grouping columns in the tlist chosen by create_plan.
* *
* This is only needed if we don't use the sub_tlist chosen by * This is only needed if we don't use the sub_tlist chosen by
* make_subplanTargetList. We have to forget the column indexes found * make_subplanTargetList. We have to forget the column indexes found