diff options
| author | Jeff Davis | 2020-06-11 18:58:16 +0000 |
|---|---|---|
| committer | Jeff Davis | 2020-06-11 19:57:43 +0000 |
| commit | 92c58fd94801dd5c81ee20e26c5bb71ad64552a8 (patch) | |
| tree | 9bcddd92b5d49527ff0cbf4af561e2ca9a0cd7c3 /src/include/optimizer | |
| parent | 5940ffb221316ab73e6fdc780dfe9a07d4221ebb (diff) | |
Rework HashAgg GUCs.
Eliminate enable_groupingsets_hash_disk, which was primarily useful
for testing grouping sets that use HashAgg and spill. Instead, hack
the table stats to convince the planner to choose hashed aggregation
for grouping sets that will spill to disk. Suggested by Melanie
Plageman.
Rename enable_hashagg_disk to hashagg_avoid_disk_plan, and invert the
meaning of on/off. The new name indicates more strongly that it only
affects the planner. Also, the word "avoid" is less definite, which
should avoid surprises when HashAgg still needs to use the
disk. Change suggested by Justin Pryzby, though I chose a different
GUC name.
Discussion: https://postgr.es/m/CAAKRu_aisiENMsPM2gC4oUY1hHG3yrCwY-fXUg22C6_MJUwQdA%40mail.gmail.com
Discussion: https://postgr.es/m/20200610021544.GA14879@telsasoft.com
Backpatch-through: 13
Diffstat (limited to 'src/include/optimizer')
| -rw-r--r-- | src/include/optimizer/cost.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/include/optimizer/cost.h b/src/include/optimizer/cost.h index 9710e5c0a45..92e70ec0d9b 100644 --- a/src/include/optimizer/cost.h +++ b/src/include/optimizer/cost.h @@ -55,8 +55,7 @@ extern PGDLLIMPORT bool enable_tidscan; extern PGDLLIMPORT bool enable_sort; extern PGDLLIMPORT bool enable_incrementalsort; extern PGDLLIMPORT bool enable_hashagg; -extern PGDLLIMPORT bool enable_hashagg_disk; -extern PGDLLIMPORT bool enable_groupingsets_hash_disk; +extern PGDLLIMPORT bool hashagg_avoid_disk_plan; extern PGDLLIMPORT bool enable_nestloop; extern PGDLLIMPORT bool enable_material; extern PGDLLIMPORT bool enable_mergejoin; |