diff options
| author | Jan Wieck <JanWieck@Yahoo.com> | 1999-05-12 15:02:39 +0000 |
|---|---|---|
| committer | Jan Wieck <JanWieck@Yahoo.com> | 1999-05-12 15:02:39 +0000 |
| commit | 79c2576f775b962c67cac136722c5c7cc98201aa (patch) | |
| tree | 3da66174208ed6b258542dde2ba9f00c0e706c07 /src/backend/optimizer/plan/planner.c | |
| parent | 1a87c14c9cf2c58009dd653c5356f68d605dff2f (diff) | |
| download | postgresql-79c2576f775b962c67cac136722c5c7cc98201aa.tar.gz postgresql-79c2576f775b962c67cac136722c5c7cc98201aa.zip | |
Replaced targetlist entry in GroupClause by reference number
in Resdom and GroupClause so changing of resno's doesn't confuse
the grouping any more.
Jan
Diffstat (limited to 'src/backend/optimizer/plan/planner.c')
| -rw-r--r-- | src/backend/optimizer/plan/planner.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c index 50d93942ec8..a55d444a8b3 100644 --- a/src/backend/optimizer/plan/planner.c +++ b/src/backend/optimizer/plan/planner.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.50 1999/05/10 00:45:20 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planner.c,v 1.51 1999/05/12 15:01:37 wieck Exp $ * *------------------------------------------------------------------------- */ @@ -260,7 +260,8 @@ union_planner(Query *parse) * belong to?) */ check_having_for_ungrouped_vars(parse->havingQual, - parse->groupClause); + parse->groupClause, + parse->targetList); } /* Calculate the opfids from the opnos */ @@ -426,8 +427,7 @@ make_subplanTargetList(Query *parse, GroupClause *grpcl = (GroupClause *) lfirst(gl); keyno++; /* sort key # for this GroupClause */ - /* Is it safe to use just resno to match tlist and glist items?? */ - if (grpcl->entry->resdom->resno == resdom->resno) + if (grpcl->tleGroupref == resdom->resgroupref) { /* Found a matching groupclause; record info for sorting */ foundGroupClause = true; |