From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-hackers(at)postgresql(dot)org, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Smolkin Grigory <smallkeen(at)gmail(dot)com> |
Subject: | Re: race condition in pg_class |
Date: | 2024-06-12 18:08:31 |
Message-ID: | CA+TgmoZdSOzXBO83qEUTe2jMQ92Ss3St8C1c59N2PPsTQ0T2Pg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jun 12, 2024 at 1:54 PM Noah Misch <noah(at)leadboat(dot)com> wrote:
> If I were making a list of changes always welcome post-beta, it wouldn't
> include adding wait event types. But I don't hesitate to add one if it
> unblocks a necessary test for a bug present in all versions.
However, injection points themselves are not present in all versions,
so even if we invent a new wait-event type, we'll have difficulty
testing older versions, unless we're planning to back-patch all of
that infrastructure, which I assume we aren't.
Personally, I think the fact that injection point wait events were put
under Extension is a design mistake that should be corrected before 17
is out of beta.
> Here's what I'm reading for each person's willingness to tolerate each option:
>
> STRATEGY | Paquier | Misch | Haas
> --------------------------------------------------------
> new "Injection Point" wait type | maybe | yes | yes
> INJECTION_POINT(...) naming | yes | yes | unknown
> isolation spec says event names | yes | no | unknown
>
> Corrections and additional strategy lines welcome. Robert, how do you judge
> the lines where I've listed you as "unknown"?
I'd tolerate INJECTION_POINT() if we had no other option but I think
it's clearly inferior. Does the last line refer to putting the
specific wait event names in the isolation spec file? If so, I'd also
be fine with that.
--
Robert Haas
EDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2024-06-12 18:11:23 | Re: On disable_cost |
Previous Message | Tom Lane | 2024-06-12 18:08:02 | Re: Proposal for Updating CRC32C with AVX-512 Algorithm. |