Re: Infinite Interval - Mailing list pgsql-hackers

From jian he
Subject Re: Infinite Interval
Date
Msg-id CACJufxHB_Otz1ctakPpvbLQ05_8i9_tDA_2e=034Kn8pmFsGUg@mail.gmail.com
Whole thread Raw
In response to Re: Infinite Interval  (jian he <jian.universality@gmail.com>)
Responses Re: Infinite Interval
List pgsql-hackers


On Wed, Jan 4, 2023 at 10:13 PM jian he <jian.universality@gmail.com> wrote:


On Tue, Jan 3, 2023 at 6:14 AM Joseph Koshakow <koshy44@gmail.com> wrote:
I have another patch, this one adds validations to operations that
return intervals and updated error messages. I tried to give all of the
error messages meaningful text, but I'm starting to think that almost all
of them should just say "interval out of range". The current approach
may reveal some implementation details and lead to confusion. For
example, some subtractions are converted to additions which would lead
to an error message about addition.

    SELECT date 'infinity' - interval 'infinity';
    ERROR:  cannot add infinite values with opposite signs

I've also updated the commit message to include the remaining TODOs,
which I've copied below

  1. Various TODOs in code.
  2. Correctly implement interval_part for infinite intervals.
  3. Test consolidation.
  4. Should we just use the months field to test for infinity?


3. Test consolidation.
I used the DO command, reduced a lot of test sql code.
I don't know how to generate an interval.out file.
I hope the format is ok. I use https://sqlformat.darold.net/ format the sql code.
Then I saw on the internet that one line should be no more than 80 chars. so I slightly changed the format.

--
 I recommend David Deutsch's <<The Beginning of Infinity>>

  Jian




1. Various TODOs in code.
logic combine and clean up for functions in backend/utils/adt/timestamp.c (timestamp_pl_interval,timestamptz_pl_interval, interval_pl, interval_mi).
3. Test consolidation in /regress/sql/interval.sql

For 1. I don't know how to format the code. I have a problem installing pg_indent. If the format is wrong, please reformat.
3. As the previous email thread.



Attachment

pgsql-hackers by date:

Previous
From: Alexander Korotkov
Date:
Subject: Re: Allow placeholders in ALTER ROLE w/o superuser
Next
From: Shruthi Gowda
Date:
Subject: Issue in MERGE with concurrent UPDATE and MERGE