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_costs.patch
Description: Binary data