Re: [HACKERS] CUBE seems a bit confused about ORDER BY

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

In response to

Responses

Browse pgsql-hackers by date

  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