On 11 October 2010 10:55, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>> BTW, why has percentile been removed from this patch? As the more
>> general, and SQL standard function, that would seem to be the more
>> useful one to include. Upthread it was mentioned that there is already
>> an ntile window function, but actually that's a completely different
>> thing.
>
> The reason for removing was impossibility to specify so some parameter
> must by immutable - in this case p parameter should be immutable
> otherwise the result is undefined.
>
Could we not just make an arbitrary choice, like the last non-null
value, and then document that. I can't believe that this would ever be
an issue in practice.
I don't think there is any way to deduce the following from behaviour
from the documentation:
select string_agg(i::text, repeat(',',i)) from generate_series(1,10) as g(i); string_agg
-------------------------------------------------------------------1,,2,,,3,,,,4,,,,,5,,,,,,6,,,,,,,7,,,,,,,,8,,,,,,,,,9,,,,,,,,,,10
(1 row)
but I don't see it as a real problem for the aggregate.
Thoughts?
Regards,
Dean