Re: wip: functions median and percentile

From: Hitoshi Harada <umi(dot)tanuki(at)gmail(dot)com>
To: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, David Fetter <david(at)fetter(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: wip: functions median and percentile
Date: 2010-10-03 14:06:25
Message-ID: AANLkTikBDCqEFaTtPX2hOKgkCrz-+FCddeBPBZavz6Rw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-rrreviewers

2010/10/2 Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>:
> Hello
>
> updated version
>  * memsort removed
>  * window aggregate support blocked

I ran this patch and it looks good for committer.
Just one thing, in src/backend/utils/adt/Makefile median.o to compile
the new file is missing.

And I'm now thinking about how to make median happen in window
aggregate. In fact we might fix parse_func.c to distinguish the same
name aggregate and window functions and median() can be implemented
separately from aggregate one. But it's another story than this patch.
For now we can commit the median aggregate only, without window
function support.

Regards,

--
Hitoshi Harada

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomoaki Sato 2010-10-03 14:46:21 pgadmin3_90 package
Previous Message Peter Geoghegan 2010-10-03 13:53:34 Re: ISN patch that applies cleanly with git apply

Browse pgsql-rrreviewers by date

  From Date Subject
Next Message Greg Stark 2010-10-03 15:49:37 Re: wip: functions median and percentile
Previous Message Hitoshi Harada 2010-10-03 13:47:28 Re: [RRR] top-level DML under CTEs