Re: Default setting for enable_hashagg_disk (hash_mem) - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Default setting for enable_hashagg_disk (hash_mem)
Date
Msg-id 20200703195002.GA28095@alvherre.pgsql
Whole thread Raw
In response to Re: Default setting for enable_hashagg_disk (hash_mem)  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
On 2020-Jul-03, Bruce Momjian wrote:

> Well, the bottom line is that we are designing features during beta.

Well, we're designing a way for users to interact the new feature.
The feature itself is already in, and it works well in general terms.  I
expect that the new behavior is a win in the majority of cases, and the
problem being discussed here will only manifest as a regression in
corner cases.  (I don't have data to back this up, but if this weren't
the case we would have realized much earlier).

It seem to me we're designing a solution to a problem that was found
during testing, which seems perfectly acceptable to me.  I don't see
grounds for reverting the behavior and I haven't seen anyone suggesting
that it would be an appropriate solution to the issue.

> If we add a new behavior to PG 13, we then have the pre-PG 13 behavior,
> the pre-patch behavior, and the post-patch behavior.  How are people
> supposed to test all of that?

They don't have to.  We tell them that we added some new tweak for a new
pg13 feature in beta3 and that's it.

> Add to that that some don't even feel we
> need a new behavior, which is delaying any patch from being applied.

If we don't need any new behavior, then we would just close the open
item and call the current state good, no?

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Inlining of couple of functions in pl_exec.c improves performance
Next
From: Alvaro Herrera
Date:
Subject: Re: Missing "Up" navigation link between parts and doc root?