From: Nathan Bossart <nathandbossart@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: pgsql-hackers@postgresql.org <pgsql-hackers@postgresql.org>
Cc: Michael Paquier <michael@paquier.xyz>
Subject: Re: verify predefined LWLocks have entries in wait_event_names.txt
Date: Tue, 2 Jan 2024 15:45:44 -0600
Message-ID: <20240102214544.GA1063425@nathanxps13> (raw)
In-Reply-To: <CA+TgmoYRhhL12iOcNKv6f__iZMk1_BPgbASFykPazhr_bwk0=w@mail.gmail.com>
References: <20240102173120.GA1061678@nathanxps13>
<CA+TgmoYRhhL12iOcNKv6f__iZMk1_BPgbASFykPazhr_bwk0=w@mail.gmail.com>
On Tue, Jan 02, 2024 at 01:13:16PM -0500, Robert Haas wrote:
> On Tue, Jan 2, 2024 at 12:31 PM Nathan Bossart <nathandbossart@gmail.com> wrote:
>> I think we're supposed to omit the "Lock" suffix in wait_event_names.txt.
>
> Ugh, sorry. But also, why in the world?
That seems to date back to commit 14a9101. I can agree that the suffix is
somewhat redundant since these are already marked as type "LWLock", but
I'll admit I've been surprised by this before, too. IMHO it makes this
proposed test more important because you can't just grep for a different
lock to find all the places you need to update.
> - Check in both directions instead of just one?
>
> - Verify ordering?
To do those things, I'd probably move the test to one of the scripts that
generates the documentation or header file (pg_wait_events doesn't tell us
whether a lock is predefined or what order it's listed in). That'd cause
failures at build time instead of during testing, which might be kind of
nice, too.
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: nathandbossart@gmail.com, robertmhaas@gmail.com, michael@paquier.xyz
Subject: Re: verify predefined LWLocks have entries in wait_event_names.txt
In-Reply-To: <20240102214544.GA1063425@nathanxps13>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox