Re: Infinite Interval

From: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>
To: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, Joseph Koshakow <koshy44(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Gregory Stark (as CFM)" <stark(dot)cfm(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Infinite Interval
Date: 2023-11-09 12:49:53
Message-ID: CAEZATCXAW+DmM2otTB7ROqHAuYok7D2Ze+hPmOzQd5Qt=QqDQw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, 9 Nov 2023 at 08:37, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> wrote:
>
> On Thu, 9 Nov 2023 at 07:15, Ashutosh Bapat
> <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> >
> > Just to test whether that bug fix also fixes the failure seen with
> > this patchset, I am attaching the patchset including the patch with
> > the fix.
> >
> > 0001 - fix in other thread
> > 0002 and 0003 are 0001 and 0002 in the previous patch set.
>
> Thanks. That's confirmed, it has indeed turned green!
>

OK, I have pushed 0001 and 0002. Here's the remaining (main) patch.

I couldn't resist making one more cosmetic change -- I moved
finite_interval_pl() and finite_interval_mi() up to where they're
first used, which is where that code was originally, making it
slightly easier to compare old-vs-new code side-by-side, and because I
think that's the more natural place for them.

Regards,
Dean

Attachment Content-Type Size
v32-0001-Support-infinity-in-the-interval-data-type.patch text/x-patch 162.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dean Rasheed 2023-11-09 13:24:49 Re: MERGE ... RETURNING
Previous Message Amul Sul 2023-11-09 12:00:49 Re: ALTER COLUMN ... SET EXPRESSION to alter stored generated column's expression