Re: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: John Brothers <johnbr(at)mindspring(dot)com>
Cc: pgsql-general(at)hub(dot)org, pgsql-sql(at)hub(dot)org
Subject: Re: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4
Date: 2000-01-28 04:49:14
Message-ID: 9485.949034954@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers pgsql-sql

John Brothers <johnbr(at)mindspring(dot)com> writes:
> I don't think that patch will work - Hiroshi whipped up that patch for
> me a week ago for a different problem - we have a table with duplicate
> primary keys, which seems to be an arithmetic overflow problem because
> the index key values can be both very large positive and very large
> negative numbers.

Actually, if Nicolas' table contains both very large positive and very
large negative integers, then his index could be messed up pretty badly.
What Hiroshi saw (and I missed :-() was that btint4cmp can fail and
return a result of the wrong sign if the difference between two integers
overflows. Since index sorting depends critically on the assumption
that the comparator always returns consistent results (a < b and b < c
must imply a < c, but this can fail if a - c overflows), you could have
an out-of-order index. And then probes into the index could fail to
find items they should find ... which is exactly the complained-of
symptom.

Hiroshi neglected to mention that you'd probably need to drop and
recreate the index after applying the patch; if it's indeed out of
order, just patching the comparator bug isn't enough to fix it.

regards, tom lane

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Bruce Momjian 2000-01-28 05:31:26 Re: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4
Previous Message Hiroshi Inoue 2000-01-28 04:40:56 RE: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2000-01-28 04:51:03 Re: [HACKERS] very minor problem in contrib/dateformat/to-from_char.c
Previous Message Bruce Momjian 2000-01-28 04:48:23 Re: OIDS (Re: [HACKERS] Well, then you keep your darn columns)

Browse pgsql-sql by date

  From Date Subject
Next Message Bruce Momjian 2000-01-28 05:31:26 Re: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4
Previous Message Hiroshi Inoue 2000-01-28 04:40:56 RE: [SQL] RE: [GENERAL] Problem with SELECT on large negative INT4