diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 1999-12-09 05:58:56 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 1999-12-09 05:58:56 +0000 |
| commit | f7f41c7c8c34edec610240405569ed6e0954e03e (patch) | |
| tree | 39c0737d7aceed48126cf0f20da82fd4c2442b5b /src/backend/optimizer/plan/planmain.c | |
| parent | d65a27f9509e8cbd0a0d8a58597bd096c2a22207 (diff) | |
Replace generic 'Illegal use of aggregates' error message with one that
shows the specific ungrouped variable being complained of. Perhaps this
will reduce user confusion...
Diffstat (limited to 'src/backend/optimizer/plan/planmain.c')
| -rw-r--r-- | src/backend/optimizer/plan/planmain.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/backend/optimizer/plan/planmain.c b/src/backend/optimizer/plan/planmain.c index 7da2ee4c211..7dcafe35dce 100644 --- a/src/backend/optimizer/plan/planmain.c +++ b/src/backend/optimizer/plan/planmain.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planmain.c,v 1.47 1999/11/15 02:00:07 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/planmain.c,v 1.48 1999/12/09 05:58:52 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -100,10 +100,7 @@ query_planner(Query *root, * Note we do NOT do this for subplans in WHERE; it's legal * there because WHERE is evaluated pre-GROUP. */ - if (check_subplans_for_ungrouped_vars((Node *) tlist, - root->groupClause, - tlist)) - elog(ERROR, "Sub-SELECT must use only GROUPed attributes from outer SELECT"); + check_subplans_for_ungrouped_vars((Node *) tlist, root, tlist); } } |