Re: Abbreviated keys for Numeric (was: Re: B-Tree support function number 3 (strxfrm() optimization))

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