Thread: pgsql: Fix ancient typo in user-defined-aggregates documentation.
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(-)
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
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