From: Tom Lane <tgl@sss.pgh.pa.us>
To: =?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?= <me@komzpa.net>
cc: Nikita Glukhov <n.gluhov@postgrespro.ru>,
        Alexander Korotkov <aekorotkov@gmail.com>,
        pgsql-hackers <pgsql-hackers@postgresql.org>,
        Heikki Linnakangas <hlinnaka@iki.fi>, Fedor Sigaev <teodor@sigaev.ru>
Subject: Re: compress method for spgist - 2
In-reply-to: 
 <CAC8Q8tJ6jPJd_BJftq9CVrUNTNM__P-H4jbNO9qr5oV=bYJU0w@mail.gmail.com>
References: <54907069.1030506@sigaev.ru>
 <CAPpHfdsoy2bJMSTtvQySJ9zN2YJmzWJBLvdHV=ZvMOm6BR0UFg@mail.gmail.com>
 <20170920190059.1354.29457.pgcf@coridan.postgresql.org>
 <31999.1505938053@sss.pgh.pa.us>
 <CAPpHfduv0XZ_k+D3NoFMr=wFVM22D=wFZCMMswoLwBe2jAbuJw@mail.gmail.com>
 <60e048d3-aa29-8289-5438-3b0c4c1d8d84@postgrespro.ru>
 <CAC8Q8tJ6jPJd_BJftq9CVrUNTNM__P-H4jbNO9qr5oV=bYJU0w@mail.gmail.com>
Comments: In-reply-to =?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?=
 <me@komzpa.net>
	message dated "Wed, 20 Sep 2017 23:06:30 -0000"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <7348.1505949640.1@sss.pgh.pa.us>
Date: Wed, 20 Sep 2017 19:20:40 -0400
Message-ID: <7349.1505949640@sss.pgh.pa.us>
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

=?UTF-8?Q?Darafei_=22Kom=D1=8Fpa=22_Praliaskouski?= <me@komzpa.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


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers