Skip site navigation (1) Skip section navigation (2)

Re: wip: functions median and percentile

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Hitoshi Harada <umi(dot)tanuki(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 17:31:18
Message-ID: AANLkTikT-QExuaxW-6YMYHjM+UVkc9yNU2CrZfgHWOrX@mail.gmail.com (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-rrreviewers
Hello

2010/10/3 Hitoshi Harada <umi(dot)tanuki(at)gmail(dot)com>:
> 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,
>

Thank you very much for review

fixed patch + missing Makefile

Regards

Pavel Stehule
>
> --
> Hitoshi Harada
>

Attachment: median.diff
Description: application/octet-stream (17.2 KB)

In response to

pgsql-hackers by date

Next:From: Peter GeogheganDate: 2010-10-03 20:37:43
Subject: Re: pgadmin
Previous:From: Bernd HelmleDate: 2010-10-03 17:05:42
Subject: ALTER DATABASE RENAME with HS/SR

pgsql-rrreviewers by date

Next:From: Hitoshi HaradaDate: 2010-10-04 06:36:48
Subject: Re: wip: functions median and percentile
Previous:From: Greg StarkDate: 2010-10-03 15:49:37
Subject: Re: wip: functions median and percentile

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group