Re: BRIN minmax multi - incorrect distance for infinite timestamp/date

From: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>
To: Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>
Cc: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: BRIN minmax multi - incorrect distance for infinite timestamp/date
Date: 2023-10-18 10:56:39
Message-ID: CAEZATCULM5CkfuRRi1k1Ad=yAbW707M-gt9NJCYvR35s89taEQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, 18 Oct 2023 at 09:16, Tomas Vondra
<tomas(dot)vondra(at)enterprisedb(dot)com> wrote:
>
> BTW when adding the tests with extreme values, I noticed this:
>
> test=# select '5874897-01-01'::date;
> date
> ---------------
> 5874897-01-01
> (1 row)
>
> test=# select '5874897-01-01'::date + '1 second'::interval;
> ERROR: date out of range for timestamp
>

That's correct because date + interval returns timestamp, and the
value is out of range for a timestamp. This is equivalent to:

select '5874897-01-01'::date::timestamp + '1 second'::interval;
ERROR: date out of range for timestamp

and I think it's good that it gives a different error from this:

select '294276-01-01'::date::timestamp + '1 year'::interval;
ERROR: timestamp out of range

so you can tell that the overflow in the first case happens before the addition.

Of course a side effect of internally casting first is that you can't
do things like this:

select '5874897-01-01'::date - '5872897 years'::interval;
ERROR: date out of range for timestamp

which arguably ought to return '2000-01-01 00:00:00'. In practice
though, I think it would be far more trouble than it's worth trying to
change that.

Regards,
Dean

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2023-10-18 11:02:11 Re: LLVM 16 (opaque pointers)
Previous Message Amit Kapila 2023-10-18 10:54:10 Re: Synchronizing slots from primary to standby