pgsql: Fix incorrect comment for get_agg_clause_costs - Mailing list pgsql-committers

From David Rowley
Subject pgsql: Fix incorrect comment for get_agg_clause_costs
Date
Msg-id E1m7qmc-0007A2-QM@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix incorrect comment for get_agg_clause_costs

Adjust the header comment in get_agg_clause_costs so that it matches what
the function currently does.  No recursive searching has been done ever
since 0a2bc5d61.  It also does not determine the aggtranstype like the
comment claimed. That's all done in preprocess_aggref().
preprocess_aggref also now determines the numOrderedAggs, so remove the
mention that get_agg_clause_costs also calculates "counts".

Normally, since this is just an adjustment of a comment it might not be
worth back-patching, but since this code is new to PG14 and that version
is still in beta, then it seems worth having the comments match.

Discussion: https://postgr.es/m/CAApHDvrrGrTJFPELrjx0CnDtz9B7Jy2XYW3Z2BKifAWLSaJYwQ@mail.gmail.com
Backpatch-though: 14

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/2b58f894e56a1944e824e19c92337d6bf24d9c41

Modified Files
--------------
src/backend/optimizer/prep/prepagg.c | 28 ++++++++++++----------------
1 file changed, 12 insertions(+), 16 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Fix a couple of memory leaks in src/bin/pg_basebackup/
Next
From: David Rowley
Date:
Subject: pgsql: Fix incorrect comment for get_agg_clause_costs