Re: [HACKERS] compress method for spgist - 2 - Mailing list pgsql-hackers

From Darafei "Komяpa" Praliaskouski
Subject Re: [HACKERS] compress method for spgist - 2
Date
Msg-id CAC8Q8tJ6jPJd_BJftq9CVrUNTNM__P-H4jbNO9qr5oV=bYJU0w@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] compress method for spgist - 2  (Nikita Glukhov <n.gluhov@postgrespro.ru>)
Responses Re: [HACKERS] compress method for spgist - 2
Re: [HACKERS] compress method for spgist - 2
List pgsql-hackers
It is possible for bbox->low.x to be NaN when circle->center.x is and
circle->radius are both +Infinity. 

What is rationale behind this circle? 
It seems to me that any circle with radius of any Infinity should become a [-Infinity .. Infinity, -Infinity .. Infinity] box. Then you won't have NaNs, and index structure shouldn't be broken. 

If it happens because NaN > Infinity, then the check should be not for isnan, but for if (low>high){swap(high, low)}. It probably should be a part of box, not a part of circle, maths.

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [HACKERS] CREATE COLLATION does not sanitize ICU's BCP 47language tags. Should it?
Next
From: Peter Geoghegan
Date:
Subject: Re: [HACKERS] CREATE COLLATION does not sanitize ICU's BCP 47language tags. Should it?