Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST
Date
Msg-id 2608.1568924017@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST  (Erik Rijkers <er@xs4all.nl>)
Responses Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
List pgsql-committers
Erik Rijkers <er@xs4all.nl> writes:
> Oops:
>       create_index                 ... ok          634 ms
>       create_index_spgist          ... FAILED      438 ms
>       create_view                  ... ok          329 ms

I'm betting the issue is breaking the Datum abstraction here:

-                   scan->xs_orderbyvals[i] = Float8GetDatum(distanceValues[i]);
+                   scan->xs_orderbyvals[i] = item->distances[i].value;

AFAICS, item->distances[i].value is a double not a Datum, so dropping
the Float8GetDatum call is just wrong.

            regards, tom lane



pgsql-committers by date:

Previous
From: Erik Rijkers
Date:
Subject: Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST
Next
From: Alexander Korotkov
Date:
Subject: Re: pgsql: Improve handling of NULLs in KNN-GiST and KNN-SP-GiST