Re: enable_timeout_every() and fin_time - Mailing list pgsql-hackers

From Robert Haas
Subject Re: enable_timeout_every() and fin_time
Date
Msg-id CA+TgmoZvsehoTLyOodQeFvQsW85Yg_Rt3LSzXksYW_15oQs6Zw@mail.gmail.com
Whole thread Raw
In response to Re: enable_timeout_every() and fin_time  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
On Tue, Jan 3, 2023 at 3:14 PM Andres Freund <andres@anarazel.de> wrote:
> Doesn't that discrepancy already exist as the code stands, because
> startup_progress_phase_start_time is also set in
> has_startup_progress_timeout_expired()?

I don't think it is, actually.

> I realize that was an example, but the
> issue seems broader: After the first "firing", the next timeout will be
> computed relative to an absolute time gathered in timestamp.c.

We're computing the time since the start of the current phase, not the
time since the last timeout. So I don't see how this is relevant.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Why are we using blocking libpq in the backend?
Next
From: Brar Piening
Date:
Subject: Re: doc: add missing "id" attributes to extension packaging page