Re: Eager aggregation, take 3 - Mailing list pgsql-hackers

From Richard Guo
Subject Re: Eager aggregation, take 3
Date
Msg-id CAMbWs48TAaXA=8B47yi6gJcufNeig05cMrXer4tyqEGKu83qpA@mail.gmail.com
Whole thread Raw
In response to Re: Eager aggregation, take 3  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Eager aggregation, take 3
List pgsql-hackers
On Wed, Oct 8, 2025 at 11:45 PM Robert Haas <robertmhaas@gmail.com> wrote:
> On Tue, Oct 7, 2025 at 6:57 AM Richard Guo <guofenglinux@gmail.com> wrote:
> > I retained the name of this field though.

> For what it's worth, I also don't like that field name. I'm not sure
> what to propose instead, but I don't think apply_at is very clear.

This field represents the set of relids at which partial aggregation
is applied.  So how about naming it partial_agg_designated_relids?
That feels a bit verbose, though.  How about partial_agg_relids or,
for brevity, agg_relids instead?

- Richard



pgsql-hackers by date:

Previous
From: Richard Guo
Date:
Subject: Re: Eager aggregation, take 3
Next
From: David Rowley
Date:
Subject: Re: [PATCH] Add tests for Bitmapset