diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2003-06-16 02:03:38 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2003-06-16 02:03:38 +0000 |
| commit | a4997254693057d4587d74222881b8e03580da2c (patch) | |
| tree | c022e641ec691ce3f108b28e0720444da6ed3eef /src/backend/optimizer/plan/createplan.c | |
| parent | cb02610e503957d7ed9b4375537fb6275c16f1fa (diff) | |
| download | postgresql-a4997254693057d4587d74222881b8e03580da2c.tar.gz postgresql-a4997254693057d4587d74222881b8e03580da2c.zip | |
Allow GROUP BY, ORDER BY, DISTINCT targets to be unknown literals,
silently resolving them to type TEXT. This is comparable to what we
do when faced with UNKNOWN in CASE, UNION, and other contexts. It gets
rid of this and related annoyances:
select distinct f1, '' from int4_tbl;
ERROR: Unable to identify an ordering operator '<' for type unknown
This was discussed many moons ago, but no one got round to fixing it.
Diffstat (limited to 'src/backend/optimizer/plan/createplan.c')
| -rw-r--r-- | src/backend/optimizer/plan/createplan.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/optimizer/plan/createplan.c b/src/backend/optimizer/plan/createplan.c index 273a80129eb..81ee7962df9 100644 --- a/src/backend/optimizer/plan/createplan.c +++ b/src/backend/optimizer/plan/createplan.c @@ -10,7 +10,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v 1.145 2003/06/15 22:51:45 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/createplan.c,v 1.146 2003/06/16 02:03:37 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -583,7 +583,7 @@ create_unique_plan(Query *root, UniquePath *best_path) { List *sortList; - sortList = addAllTargetsToSortList(NIL, my_tlist); + sortList = addAllTargetsToSortList(NULL, NIL, my_tlist, false); plan = (Plan *) make_sort_from_sortclauses(root, my_tlist, subplan, sortList); plan = (Plan *) make_unique(my_tlist, plan, sortList); |