On Wed, Oct 8, 2025 at 11:45 PM Robert Haas <[email protected]> wrote: > On Tue, Oct 7, 2025 at 6:57 AM Richard Guo <[email protected]> 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
