On 14.04.23 12:03, Dagfinn Ilmari Mannsåker wrote:
> Thanks for the heads-up, here's a rebased patch. I've also formatted
> the lines to match what reformat_dat_file.pl wants. It also wanted to
> reformat a bunch of other entries, but I left those alone.
>
> - ilmari
The patch applies cleanly now and \df shows the argument names:
postgres=# \df string_agg
List of functions
Schema | Name | Result data type | Argument data
types | Type
------------+------------+------------------+------------------------------+------
pg_catalog | string_agg | bytea | value bytea, delimiter
bytea | agg
pg_catalog | string_agg | text | value text, delimiter
text | agg
(2 rows)
postgres=# \df json_object_agg
List of functions
Schema | Name | Result data type | Argument data
types | Type
------------+-----------------+------------------+------------------------+------
pg_catalog | json_object_agg | json | key "any", value
"any" | agg
(1 row)
I'm wondering if there are some sort of guidelines that dictate when to
name an argument or not. It would be nice to have one for future reference.
I will mark the CF entry as "Read for Committer" and let the committers
decide if it's best to first create a guideline for that or not.
Best, Jim