| 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 |
| 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 |