diff options
| author | Peter Geoghegan <pg@bowt.ie> | 2020-07-28 17:14:06 -0700 |
|---|---|---|
| committer | Peter Geoghegan <pg@bowt.ie> | 2020-07-28 17:14:06 -0700 |
| commit | b6c15e71f33fe9aa7f38cc7bde26d420fbaaef5b (patch) | |
| tree | 901140e24b508b52024f7f445b36d4a29ef8f5ea | |
| parent | e362f469c50f6e671285640cc2087345ab55a9b2 (diff) | |
| download | postgresql-b6c15e71f33fe9aa7f38cc7bde26d420fbaaef5b.tar.gz postgresql-b6c15e71f33fe9aa7f38cc7bde26d420fbaaef5b.zip | |
Correct obsolete UNION hash aggs comment.
Oversight in commit 1f39bce0, which added disk-based hash aggregation.
Backpatch: 13-, where disk-based hash aggregation was introduced.
| -rw-r--r-- | src/backend/optimizer/prep/prepunion.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/backend/optimizer/prep/prepunion.c b/src/backend/optimizer/prep/prepunion.c index 951aed80e7a..6588f83d5ec 100644 --- a/src/backend/optimizer/prep/prepunion.c +++ b/src/backend/optimizer/prep/prepunion.c @@ -944,11 +944,10 @@ make_union_unique(SetOperationStmt *op, Path *path, List *tlist, /* * XXX for the moment, take the number of distinct groups as equal to the - * total input size, ie, the worst case. This is too conservative, but we - * don't want to risk having the hashtable overrun memory; also, it's not - * clear how to get a decent estimate of the true size. One should note - * as well the propensity of novices to write UNION rather than UNION ALL - * even when they don't expect any duplicates... + * total input size, ie, the worst case. This is too conservative, but + * it's not clear how to get a decent estimate of the true size. One + * should note as well the propensity of novices to write UNION rather + * than UNION ALL even when they don't expect any duplicates... */ dNumGroups = path->rows; |