From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
---|---|
To: | Justin Pryzby <pryzby(at)telsasoft(dot)com>, Melanie Plageman <melanieplageman(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Default setting for enable_hashagg_disk |
Date: | 2020-06-11 20:22:57 |
Message-ID: | bfec56be9bd5e245da4e733197af9b485bf40362.camel@j-davis.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs pgsql-hackers |
On Wed, 2020-06-10 at 11:39 -0700, Jeff Davis wrote:
> 1. Testing the spilling of hashed grouping sets: I'm inclined to just
> get rid of enable_groupingsets_hash_disk and use Melanie's stats-
> hacking approach instead.
Fixed in 92c58fd9.
> think the names you suggested quite fit, but the idea to use a more
> interesting GUC value might help express the behavior. Perhaps making
> enable_hashagg a ternary "enable_hashagg=on|off|avoid_disk"? The word
> "reject" is too definite for the planner, which is working with
> imperfect information.
I renamed enable_hashagg_disk to hashagg_avoid_disk_plan, which I think
satisfies the concerns raised here. Also in 92c58fd9.
There is still the original topic of this thread, which is whether we
need to change the default value of this GUC, or penalize disk-based
HashAgg in some way, to be more conservative about plan changes in v13.
I think we can wait a little longer to make a decision there.
Regards,
Jeff Davis
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2020-06-11 22:38:36 | Re: Adding xreflable |
Previous Message | Shay Rojansky | 2020-06-11 18:06:52 | Re: Signed-ness of ints is unclear in FE-BE protocol docs |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2020-06-11 20:40:54 | extensible options syntax for replication parser? |
Previous Message | Alvaro Herrera | 2020-06-11 20:19:21 | Re: [PATCH] fix two shadow vars (src/backend/commands/sequence.c) |