From: | Joseph Koshakow <koshy44(at)gmail(dot)com> |
---|---|
To: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Infinite Interval |
Date: | 2022-12-15 23:43:29 |
Message-ID: | CAAvxfHc5ZLsMAYmKaGOJVGvZyhZS+eZ-p_HZDZF4JsJYAtCj+A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Dec 12, 2022 at 8:05 AM Ashutosh Bapat
<ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
>
> Hi Joseph,
> I stumbled upon this requirement a few times. So I started working on
> this support in my spare time as a hobby project to understand
> horology code in PostgreSQL. This was sitting in my repositories for
> more than an year. Now that I have someone else showing an interest,
> it's time for it to face the world. Rebased it, fixed conflicts.
>
> PFA patch implementing infinite interval. It's still WIP, there are
> TODOs in the code and also the commit message lists things that are
> known to be incomplete. You might want to assess expected output
> carefully
That's great! I was also planning to just work on it as a hobby
project, so I'll try and review and add updates as I find free
time as well.
> > The proposed design from the most recent thread was to reserve
> > INT32_MAX months for infinity and INT32_MIN months for negative
> > infinity. As pointed out in the thread, these are currently valid
> > non-infinite intervals, but they are out of the documented range.
>
> The patch uses both months and days together to avoid this problem.
Can you expand on this part? I believe the full range of representable
intervals are considered valid as of v15.
- Joe Koshakow
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-12-15 23:47:51 | Add enable_presorted_aggregate GUC |
Previous Message | Paul Jungwirth | 2022-12-15 23:33:34 | Exclusion constraints on partitioned tables |