On 21.2.2015 01:45, Peter Geoghegan wrote:
> On Fri, Feb 20, 2015 at 4:42 PM, Tomas Vondra
> <tomas.vondra@2ndquadrant.com> wrote:
>> Isn't this patch about adding abbreviated keys for Numeric data type?
>> That's how I understood it, and looking into numeric_sortsup.patch seems
>> to confirm that.
>>
>> There's another patch for Datum, but that's a different thread.
>
> Right...so don't test a datum sort case, since that isn't supported at
> all in the master branch. Your test case is invalid for that reason.
What do you mean by 'Datum sort case'? The test I was using is this:
create table stuff as select (random())::numeric as randnum from
generate_series(1,1000000);
select percentile_disc(0) within group (order by randnum) from stuff;
That's a table with a Numeric column, and a sort on that Numeric, no?
--
Tomas Vondra http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services