Re: Add wait events for server logging destination writes

From: 신성준 <shinsj4653(at)gmail(dot)com>
To: Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Nikolay Samokhvalov <nik(at)postgres(dot)ai>, assam258(at)gmail(dot)com, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Kirk Wolak <wolakk(at)gmail(dot)com>, Andreas Karlsson <andreas(at)proxel(dot)se>, Jakub Wartak <jakub(dot)wartak(at)enterprisedb(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, jian he <jian(dot)universality(at)gmail(dot)com>
Subject: Re: Add wait events for server logging destination writes
Date: 2026-11-13 08:48:10
Message-ID: CACdN0M54iGOWob41Sy7pZM8A3bzMVoPsvodyQPFBzcMXUOZnKA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Michael Paquier wrote:
> Note that I am not convinced about adding the whole set of suggested
> wait events just based on the reason that they seem useful.

The ones in 0002 come from real cases. Jakub's outage upthread was a
stuck remote syslog (SyslogWrite). With Andrey's slow boot disk,
backends block on the syslogger pipe or on stderr, depending on
logging_collector (SysloggerWrite, StderrWrite).

> For one, I am not able to check the Windows-specific ones and if they
> actually make sense on their own.

Fair. EventlogWrite has only been seen in CI runs, and WriteConsoleW
has never been exercised. I'm fine with dropping 0003 and leaving the
Windows paths for later.

Andrey Borodin wrote:
> Could we reuse the debug nesting check to collect inner/outer call
> sites over a check-world run and identify most useful sites that can
> be converted to nested?

Yes. A run in September found only two such pairs, but the check
aborts at the first one in each process. I'll make it record the pairs
instead and post the list here.

I'll hold off on a new version until there are more opinions on the
swap approach.
--
Best regards,
Seongjun

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andreas Karlsson 2026-11-13 08:52:49 Re: Add sortsupport to money_ops, pg_lsn_ops and xid8_ops
Previous Message Andreas Karlsson 2026-11-13 08:45:34 Re: [PATCH] standby: Clarify recovery conflict comment