Re: Can can I make an injection point wait occur no more than once?

From: Noah Misch <noah(at)leadboat(dot)com>
To: Peter Geoghegan <pg(at)bowt(dot)ie>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Can can I make an injection point wait occur no more than once?
Date: 2025-07-09 03:04:55
Message-ID: 20250709030455.a7@rfd.leadboat.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jul 08, 2025 at 11:21:20AM -0400, Peter Geoghegan wrote:
> On Mon, Jul 7, 2025 at 9:53 PM Noah Misch <noah(at)leadboat(dot)com> wrote:
> > If it continues to be a problem, consider sharing the patch that's behaving
> > this way for you.
>
> Attached patch shows my current progress with the isolation test.

Nothing looks suspicious in that code.

> I also attach diff output of the FreeBSD failures. Notice that the
> line "backwards_scan_session: NOTICE: notice triggered for injection
> point lock-and-validate-new-lastcurrblkno" is completely absent from
> the test output. This absence indicates that the desired test coverage
> is totally missing on FreeBSD -- so the test is completely broken on
> FreeBSD.
>
> I ran "meson test --suite setup --suite nbtree -q --print-errorlogs"
> in a loop 500 times on my Debian workstation without seeing any
> failures. Seems stable there. Whereas the FreeBSD target hasn't even
> passed once out of more than a dozen attempts. Seems to be reliably
> broken on FreeBSD.

> -backwards_scan_session: NOTICE: notice triggered for injection point lock-and-validate-new-lastcurrblkno
> +ERROR: could not find injection point lock-and-validate-left to wake up

Agreed. Perhaps it's getting a different plan type on FreeBSD, so it's not
even reaching the INJECTION_POINT() calls? That would be consistent with
these output diffs having no ERROR from attach/detach. Some things I'd try:

- Add a plain elog(WARNING) before each INJECTION_POINT()
- Use debug_print_plan or similar to confirm the plan type

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dilip Kumar 2025-07-09 03:37:03 Re: A recent message added to pg_upgade
Previous Message shveta malik 2025-07-09 03:00:38 Re: Using failover slots for PG-non_PG logical replication