From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Fix doc patch --- pg_options_to_table() returns "setof record". |
Date: | 2011-02-26 18:28:44 |
Message-ID: | 4D69465C.6040107@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 02/26/2011 12:53 PM, Bruce Momjian wrote:
> Andrew Dunstan wrote:
>>
>> On 02/26/2011 12:27 PM, Bruce Momjian wrote:
>>> Fix doc patch --- pg_options_to_table() returns "setof record".
>> This doesn't strike me as terribly helpful. Why don't we document what
>> sort of records it returns, since we know perfectly well?:
>>
>> andrew=# \df pg_options_to_table
>> List of functions
>> Schema | Name | Result data type
>> | Argument data types | Type
>> ------------+---------------------+------------------+-------------------------------------------------------------------+--------
>> pg_catalog | pg_options_to_table | SETOF record | options_array
>> text[], OUT option_name text, OUT option_value text | normal
>> (1 row)
> Well, other functions don't mention what they return either so I tried
> to be consistent. The mention below does explain their meaning, as does
> the text next to it.
At least the description of pg_get_keywords() gives the names of the
fields, which I don't see being given for pg_options_to_table(). The
only others I see in a quick search are ts_debug(), ts_stat(),
ts_parse() and ts_token_type().
Meanwhile we seem to have lost the online developer docs at
<http://developer.postgresql.org/pgdocs/postgres/index.html> :-(
> Should we fix them all?
Possibly.
> I thought that column was just for data types.
>
I don't understand this sentence.
Incidentally, your example would possibly be better written as:
select (pg_options_to_table(reloptions)).* from pg_class;
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2011-02-26 19:18:16 | pgsql: Add a configure check for Python.h if building with python. |
Previous Message | Bruce Momjian | 2011-02-26 17:53:27 | Re: pgsql: Fix doc patch --- pg_options_to_table() returns "setof record". |