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

Started by Tom Laneover 15 years ago3 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

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(-)

#2Thom Brown
thom@linux.com
In reply to: Tom Lane (#1)
Re: pgsql: Fix ancient typo in user-defined-aggregates documentation.

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

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Thom Brown (#2)
Re: pgsql: Fix ancient typo in user-defined-aggregates documentation.

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