| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Darafei "Komяpa" Praliaskouski <me(at)komzpa(dot)net> |
| Cc: | Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Fedor Sigaev <teodor(at)sigaev(dot)ru> |
| Subject: | Re: compress method for spgist - 2 |
| Date: | 2017-09-20 23:20:40 |
| Message-ID: | 7349.1505949640@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
=?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?= <me(at)komzpa(dot)net> writes:
> If it happens because NaN > Infinity, then the check should be not for
> isnan, but for if (low>high){swap(high, low)}.
Yeah, the same idea had occurred to me. It'd still need a comment, but
at least it's slightly more apparent what we're trying to ensure.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2017-09-20 23:25:15 | Re: [Proposal] Allow users to specify multiple tables in VACUUM commands |
| Previous Message | Tom Lane | 2017-09-20 23:18:08 | Re: [Proposal] Allow users to specify multiple tables in VACUUM commands |