Richard Guo <[email protected]> writes:
> On Wed, Oct 8, 2025 at 11:45 PM Robert Haas <[email protected]> wrote:
>> 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?
I might be missing a subtlety here, but how about
"apply_aggregation_at" or "apply_partial_agg_at"?
I don't think including "relids" in the field name adds anything,
given the field's declared type and comments.
regards, tom lane
