Re: Outdated comment in get_agg_clause_costs

2021-07-25 Thread David Rowley
On Fri, 23 Jul 2021 at 02:29, David Rowley wrote: > I've attached a patch that adjusts the comment so it's more aligned to > what it now does. This was a bit more outdated than I first thought. I also removed the mention of the function setting the aggtranstype and what it mentions about also ga

Outdated comment in get_agg_clause_costs

2021-07-22 Thread David Rowley
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 fix_outdated_comment_in_get_agg_clause_co