mirror/postgresql
2
0
Fork 0
mirror of https://git.postgresql.org/git/postgresql.git synced 2025-02-23 19:39:53 +08:00
Code Issues Packages Projects Releases Wiki Activity

make_sort_from_pathkeys()'s method for choosing which of several

Browse Source 
equivalent sort expressions to use was broken: you can't just look
at the relation membership, you have to actually grovel over the
individual Vars in each expression.  I think this did work when it
was written, but it was broken by subsequent optimizations that made
join relations not propagate every single input variable upward.
Must find the Var that got propagated, not choose one at random.
Per bug report from Daniel O'Neill.
This commit is contained in:
Tom Lane 2004-02-29 17:36:48 +00:00
parent 02760606d1
commit f91bde3fb6
1 changed files with 16 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/backend/optimizer/plan/createplan.c
View File

@ -10,7 +10,7 @@
*
*
* IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v 1.157.2.2 2004/01/18 00:31:53 tgl Exp $
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v 1.157.2.3 2004/02/29 17:36:48 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@ -101,7 +101,7 @@ static MergeJoin *make_mergejoin(List *tlist,
static Sort *make_sort(Query *root, List *tlist, Plan *lefttree, int numCols,
AttrNumber *sortColIdx, Oid *sortOperators);
static Sort *make_sort_from_pathkeys(Query *root, Plan *lefttree,
Relids relids, List *pathkeys);
List *pathkeys);
/*
@ -1015,7 +1015,6 @@ create_mergejoin_plan(Query *root,
outer_plan = (Plan *)
make_sort_from_pathkeys(root,
outer_plan,
best_path->jpath.outerjoinpath->parent->relids,
best_path->outersortkeys);
}
@ -1025,7 +1024,6 @@ create_mergejoin_plan(Query *root,
inner_plan = (Plan *)
make_sort_from_pathkeys(root,
inner_plan,
best_path->jpath.innerjoinpath->parent->relids,
best_path->innersortkeys);
}
@ -1793,7 +1791,6 @@ add_sort_column(AttrNumber colIdx, Oid sortOp,
* Create sort plan to sort according to given pathkeys
*
* 'lefttree' is the node which yields input tuples
* 'relids' is the set of relids represented by the input node
* 'pathkeys' is the list of pathkeys by which the result is to be sorted
*
* We must convert the pathkey information into arrays of sort key column
@ -1806,8 +1803,7 @@ add_sort_column(AttrNumber colIdx, Oid sortOp,
* adding a Result node just to do the projection.
*/
static Sort *
make_sort_from_pathkeys(Query *root, Plan *lefttree,
Relids relids, List *pathkeys)
make_sort_from_pathkeys(Query *root, Plan *lefttree, List *pathkeys)
{
List *tlist = lefttree->targetlist;
List *sort_tlist;
@ -1852,13 +1848,23 @@ make_sort_from_pathkeys(Query *root, Plan *lefttree,
}
if (!resdom)
{
/* No matching Var; look for an expression */
/* No matching Var; look for a computable expression */
foreach(j, keysublist)
{
List *exprvars;
List *k;
pathkey = lfirst(j);
if (bms_is_subset(pull_varnos(pathkey->key), relids))
exprvars = pull_var_clause(pathkey->key, false);
foreach(k, exprvars)
{
if (!tlist_member(lfirst(k), tlist))
break;
}
freeList(exprvars);
if (!k)
break; /* found usable expression */
}
if (!j)
elog(ERROR, "could not find pathkey item to sort");