Re: Infinite Interval

From: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
To: Joseph Koshakow <koshy44(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Gregory Stark (as CFM)" <stark(dot)cfm(at)gmail(dot)com>, jian he <jian(dot)universality(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Infinite Interval
Date: 2023-03-27 13:42:35
Message-ID: CAExHW5tOQOW1HyQ7gFN6bhPq8a2RanCMxq5CJ5v1vR+VmZF1JQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Mar 25, 2023 at 9:13 PM Joseph Koshakow <koshy44(at)gmail(dot)com> wrote:
>
> On Fri, Mar 24, 2023 at 9:43 AM Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> >
> > You don't need to do this, but looks like we can add DAYS_PER_WEEK macro and
> > use it here.
>
> I've attached a patch with this new macro. There's probably tons of
> places it can be used instead of hardcoding the number 7, but I'll save
> that for a future patch.

Thanks. Yes, changing other existing usages is out of scope for this patch.

Looks good to me.

--
Best Wishes,
Ashutosh Bapat

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message parth ratra 2023-03-27 13:42:42 facing issues in downloading of packages in pgadmin4
Previous Message Aleksander Alekseev 2023-03-27 13:35:50 Re: [EXTERNAL] Support load balancing in libpq