Correct obsolete UNION hash aggs comment.
authorPeter Geoghegan <pg@bowt.ie>
Wed, 29 Jul 2020 00:14:07 +0000 (17:14 -0700)
committerPeter Geoghegan <pg@bowt.ie>
Wed, 29 Jul 2020 00:14:07 +0000 (17:14 -0700)
Oversight in commit 1f39bce0, which added disk-based hash aggregation.

Backpatch: 13-, where disk-based hash aggregation was introduced.

src/backend/optimizer/prep/prepunion.c

index 951aed80e7a20df8991bb50f0a33898b971d2712..6588f83d5ec6fde6491db599b12565ac736ec8bc 100644 (file)
@@ -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;