| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
| Subject: | Outdated comment in get_agg_clause_costs |
| Date: | 2021-07-22 14:29:50 |
| Message-ID: | CAApHDvrrGrTJFPELrjx0CnDtz9B7Jy2XYW3Z2BKifAWLSaJYwQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I noticed that get_agg_clause_costs still claims that it recursively
finds Aggrefs in the expression tree, but I don't think that's been
true since 0a2bc5d61.
I've attached a patch that adjusts the comment so it's more aligned to
what it now does.
David
| Attachment | Content-Type | Size |
|---|---|---|
| fix_outdated_comment_in_get_agg_clause_costs.patch | application/octet-stream | 1.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dilip Kumar | 2021-07-22 14:31:52 | Re: [BUG]Update Toast data failure in logical replication |
| Previous Message | Tom Lane | 2021-07-22 14:24:05 | Re: printf %s with NULL pointer (was Re: BUG #17098: Assert failed on composing an error message when adding a type to an extension being dropped) |