pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
Median missing from aggregate functions page
2+ messages / 2 participants
[nested] [flat]

Commits from this thread:
  2016-12-23 3c9d398484 a377c8d921 a15c59d196 090a3870aa  Doc: improve index entry for "median".
* Median missing from aggregate functions page
@ 2016-12-23 10:20  steven.winfield@cantabcapital.com
  0 siblings, 1 reply; 2+ messages in thread

From: steven.winfield@cantabcapital.com @ 2016-12-23 10:20 UTC (permalink / raw)
  To: pgsql-docs

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/9.6/static/functions-aggregate.html
Description:

The index contains an entry for "median" on the aggregate functions page,
but there is no mention of it there.

-- 
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs


^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* Re: Median missing from aggregate functions page
@ 2016-12-23 15:21  Tom Lane <tgl@sss.pgh.pa.us>
  parent: steven.winfield@cantabcapital.com
  0 siblings, 0 replies; 2+ messages in thread

From: Tom Lane @ 2016-12-23 15:21 UTC (permalink / raw)
  To: steven.winfield@cantabcapital.com; +Cc: pgsql-docs

steven.winfield@cantabcapital.com writes:
> The index contains an entry for &quot;median&quot; on the aggregate functions page,
> but there is no mention of it there.

Hm --- that <indexterm> entry is attached to the percentile_cont()
function, which can be used to implement median, but I agree that
a person who already knew that would probably not be following the
index pointer.

It might be more useful to point to 4.2.7. Aggregate Expressions
which contains a concrete example of how to implement median
with percentile_disc().

Another idea is to just make the index entry say
"See percentile_cont() and percentile_disc()".

			regards, tom lane


-- 
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs



^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2016-12-23 15:21 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2016-12-23 10:20 Median missing from aggregate functions page steven.winfield@cantabcapital.com
2016-12-23 15:21 ` Tom Lane <tgl@sss.pgh.pa.us>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox