Rework HashAgg GUCs.
authorJeff Davis <jdavis@postgresql.org>
Thu, 11 Jun 2020 18:58:16 +0000 (11:58 -0700)
committerJeff Davis <jdavis@postgresql.org>
Thu, 11 Jun 2020 19:57:43 +0000 (12:57 -0700)
commit92c58fd94801dd5c81ee20e26c5bb71ad64552a8
tree9bcddd92b5d49527ff0cbf4af561e2ca9a0cd7c3
parent5940ffb221316ab73e6fdc780dfe9a07d4221ebb
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
doc/src/sgml/config.sgml
src/backend/optimizer/path/costsize.c
src/backend/optimizer/plan/planner.c
src/backend/utils/misc/guc.c
src/include/optimizer/cost.h
src/test/regress/expected/aggregates.out
src/test/regress/expected/groupingsets.out
src/test/regress/expected/sysviews.out
src/test/regress/sql/aggregates.sql
src/test/regress/sql/groupingsets.sql