2010/10/2 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> Hitoshi Harada <umi(dot)tanuki(at)gmail(dot)com> writes:
>> 2010/10/2 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
>>> The implementation I would've expected to see is to do the sort and then
>>> have two code paths for retrieving the median, depending on whether the
>>> sort result is all in memory or not.
>
>> Hm? The problem we encountered in the middle of the patch is there is
>> no chance to call tuplesort_end if median is called in moving frame
>> window aggregate because final function is called multiple times
>> during moving.
>
> Well, if you haven't got a solution for that, then this patch isn't
> ready for prime time.
>
> It's entirely possible that median as a window function is intractable.
> I'd rather have it throwing error than offer an implementation that will
> fall over as soon as the window gets large.

Well, that sounds like the conclusion. It is a shame, but we have to
throw an error from median() in the window aggregate, if Pavel does
not have any better solution. And as an aggregate function only, the
patch is ready if the window-related parts are removed.

Regards,


-- 
Hitoshi Harada

In response to

• Re: wip: functions median and percentile at 2010-10-01 15:22:46 from Tom Lane

Responses

• Re: wip: functions median and percentile at 2010-10-01 17:37:03 from Pavel Stehule

pgsql-hackers by date

Next:	From: Hitoshi Harada	Date: 2010-10-01 15:35:03
	Subject: Re: wip: functions median and percentile
Previous:	From: Tom Lane	Date: 2010-10-01 15:27:25
	Subject: So git pull is shorthand for what exactly?

pgsql-rrreviewers by date

Next:	From: Hitoshi Harada	Date: 2010-10-01 15:35:03
	Subject: Re: wip: functions median and percentile
Previous:	From: Tom Lane	Date: 2010-10-01 15:22:46
	Subject: Re: wip: functions median and percentile