| From: | Kirill Reshke <reshkekirill(at)gmail(dot)com> |
|---|---|
| To: | Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> |
| Cc: | surya poondla <suryapoondla4(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Reporting WAL replay progress during pre-consistency standby reovery |
| Date: | 2026-11-09 08:04:57 |
| Message-ID: | CALdSSPiQhQ9AtbsGqjTWBa9ynNkur82XZT-OdK_MmyEkzQ_75A@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, 9 Nov 2026 at 10:11, Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> wrote:
>
> > v1 applies cleanly to current master and builds without warnings.
> >
> > I think this is worth having and have a few comments:
>
> Thanks for supporting the proposal, reviewing the patch, and catching
> these issues. I appreciate the detailed analysis and testing.
>
> I have attached v2 with both changes. I restored the unconditional
> disable_startup_progress_timeout() call in EnableStandbyMode() and
> removed the log_startup_progress_interval == 0 check from
> disable_startup_progress_timeout().
>
> I also tested reloading the interval from one second to zero during
> pre-consistency replay and confirmed that the timer stops firing after
> consistency.
>
> Please review the v2 patch and let me know if you have any other comments.
>
> Best Regards,
> Nitin Jadhav
> Microsoft
Hi!
During recovery, if you try to connect to the database via psql, you
will get an error message: "the database system is not yet accepting
connection"/"Consistent recovery state has not been yet reached." etc.
What if we report the current replay LSN here?
--
Best regards,
Kirill Reshke
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nazir Bilal Yavuz | 2026-11-09 08:05:18 | Re: ci: Follow-up CI patches |
| Previous Message | Fujii Masao | 2026-11-09 07:56:18 | Re: Postgres fdw: Bug in InitPgFdwOptions initialization can cause an esoteric crash |