From: | Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> |
---|---|
To: | Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Stas Kelvich <stas(dot)kelvich(at)gmail(dot)com>, Teodor Sigaev <teodor(at)sigaev(dot)ru> |
Subject: | Re: [HACKERS] CUBE seems a bit confused about ORDER BY |
Date: | 2017-11-20 22:07:37 |
Message-ID: | CAPpHfduOeV_Kw7jhJrB1X1iEWukjnSAoTni__3KQYC0Kf+fF_A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Nov 20, 2017 at 1:59 PM, Alexander Korotkov <
a(dot)korotkov(at)postgrespro(dot)ru> wrote:
> On Mon, Nov 20, 2017 at 4:09 AM, Tomas Vondra <
> tomas(dot)vondra(at)2ndquadrant(dot)com> wrote:
>
>> Seems fine to me, although perhaps it should be split into two parts.
>> One with the cube_coord_llur fixes, and then g_cube_distance changes
>> adding support for negative coordinates.
>
>
> Thank you for your feedback. I'll split this patch into two.
>
Please, find two patches attached.
------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
Attachment | Content-Type | Size |
---|---|---|
0001-cube-knn-fix.patch | application/octet-stream | 45.9 KB |
0002-cube-knn-negative-coordinate.patch | application/octet-stream | 15.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2017-11-20 22:15:14 | Re: [HACKERS] pgbench regression test failure |
Previous Message | Alik Khilazhev | 2017-11-20 21:58:39 | Re: [HACKERS] [WIP] Zipfian distribution in pgbench |