Re: Add enable_groupagg GUC parameter to control GroupAggregate usage

From: Richard Guo <guofenglinux(at)gmail(dot)com>
To: Tatsuro Yamada <tatsuro(dot)yamada(at)ntt(dot)com>
Cc: Tatsuro Yamada <yamatattsu(at)gmail(dot)com>, David Rowley <dgrowleyml(at)gmail(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Add enable_groupagg GUC parameter to control GroupAggregate usage
Date: 2026-07-07 06:47:21
Message-ID: CAMbWs49RCumHErWYOLm4jDcvLSGqhSm1Asnwsv9Tdn2Yjn0r2A@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jun 29, 2026 at 7:02 PM Tatsuro Yamada <tatsuro(dot)yamada(at)ntt(dot)com> wrote:
> Since the enable_groupagg parameter also affects SetOp, Unique, and
> other nodes, I've created documentation patches to clarify this. I'm attaching
> them to this email. They apply on top of your v4 patch.

I've thought it over, and I'm inclined to leave the docs as they are.

My main concern is consistency with the surrounding parameters. All
of the other enable_* GUCs are documented conceptually rather than by
the specific node types they affect, and none of them enumerate the
nodes you'd see in EXPLAIN. Unique and SetOp are really EXPLAIN
labels rather than user-facing concepts, and I'd argue the existing
wording already covers them, since sort-based Unique and sorted SetOp
are both forms of sort-based grouping.

Similarly, enable_hashagg has covered hash-based SetOp for a long time
without us documenting it, which is the same convention at work. I'd
rather not add that now just to match the new parameter.

Attached is a rebased patch to make cfbot work again.

- Richard

Attachment Content-Type Size
v5-0001-Add-an-enable_groupagg-GUC-parameter.patch application/octet-stream 33.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Richard Guo 2026-07-07 06:53:20 Re: Add enable_groupagg GUC parameter to control GroupAggregate usage
Previous Message Shlok Kyal 2026-07-07 06:36:48 Re: Support EXCEPT for ALL SEQUENCES publications