Обсуждение: pgsql: Fix ancient typo in user-defined-aggregates documentation.

Поиск
Список
Период
Сортировка

pgsql: Fix ancient typo in user-defined-aggregates documentation.

От
Tom Lane
Дата:
Fix ancient typo in user-defined-aggregates documentation.

The description of the initcond value for the built-in avg(float8)
aggregate has been wrong since it was written.  Noted by Disc Magnet.

Branch
------
REL8_2_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/24cdee8606086699a634f424333b894509c5cbd0

Modified Files
--------------
doc/src/sgml/xaggr.sgml |    2 +-
1 files changed, 1 insertions(+), 1 deletions(-)


Re: pgsql: Fix ancient typo in user-defined-aggregates documentation.

От
Thom Brown
Дата:
On 23 March 2011 16:34, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Fix ancient typo in user-defined-aggregates documentation.
>
> The description of the initcond value for the built-in avg(float8)
> aggregate has been wrong since it was written.  Noted by Disc Magnet.
>
> Branch
> ------
> REL8_2_STABLE
>
> Details
> -------
> http://git.postgresql.org/pg/commitdiff/24cdee8606086699a634f424333b894509c5cbd0
>
> Modified Files
> --------------
> doc/src/sgml/xaggr.sgml |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

The preceding paragraph says that it typically uses a two-element
array for the state value.  Maybe this should say two or three-element
given the example that immediately follows it.

--
Thom Brown
Twitter: @darkixion
IRC (freenode): dark_ixion
Registered Linux user: #516935

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Re: pgsql: Fix ancient typo in user-defined-aggregates documentation.

От
Tom Lane
Дата:
Thom Brown <thom@linux.com> writes:
> On 23 March 2011 16:34, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Fix ancient typo in user-defined-aggregates documentation.
>>
>> The description of the initcond value for the built-in avg(float8)
>> aggregate has been wrong since it was written. �Noted by Disc Magnet.

> The preceding paragraph says that it typically uses a two-element
> array for the state value.  Maybe this should say two or three-element
> given the example that immediately follows it.

Oh, I shouldn't have done that with blinders on :-(.  I think probably
we have to explain that float8_accum supports more things than just
average.  Will fix as soon as I get back from my doctors appointment.

            regards, tom lane