From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
Cc: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Fix picksplit with nan values |
Date: | 2013-11-08 17:58:05 |
Message-ID: | 22236.1383933485@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alexander Korotkov <aekorotkov(at)gmail(dot)com> writes:
>> Thanks, Andrew! Good spot.
>> I didn't examine order by operators for work with NaNs.
>> I think this time problem is in GiST itself rather than in opclass. I'm
>> going to fix it in a separate patch.
> Attached patch fixes knn GiST behaviour with NaN. It makes RB-tree
> comparison function in GiST work like float8 btree opclass comparison
> function.
Hmm ... does that really work, or even do anything? I'd have thought
that if either input is a NAN, the initial test
if (sa->distances[i] != sb->distances[i])
would return false so we'd not enter the rest of it.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2013-11-08 18:19:53 | Re: pgsql: Fix blatantly broken record_image_cmp() logic for pass-by-value |
Previous Message | Josh Berkus | 2013-11-08 17:44:32 | Re: Changing pg_dump default file format |