From: | Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> |
---|---|
To: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> |
Cc: | Peter Geoghegan <pg(at)heroku(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Abbreviated keys for Numeric |
Date: | 2015-02-21 01:06:34 |
Message-ID: | 54E7DA1A.5030005@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 21.2.2015 02:00, Andrew Gierth wrote:
>>>>>> "Tomas" == Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> writes:
>
> >> 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.
>
> Tomas> What do you mean by 'Datum sort case'?
>
> A case where the code path goes via tuplesort_begin_datum rather than
> tuplesort_begin_heap.
>
> Tomas> The test I was using is this:
>
> Tomas> select percentile_disc(0) within group (order by randnum) from stuff;
>
> Sorting single columns in aggregate calls uses the Datum sort path (in
> fact I think it's currently the only place that does).
>
> Do that test with _both_ the Datum and Numeric sort patches in place,
> and you will see the effect. With only the Numeric patch, the numeric
> abbrev code is not called.
D'oh! Thanks for the explanation.
--
Tomas Vondra http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2015-02-21 01:46:31 | Re: Bootstrap DATA is a pita |
Previous Message | Andrew Gierth | 2015-02-21 01:00:42 | Re: Abbreviated keys for Numeric |