Re: Stabilize recovery conflict stats checks in 031_recovery_conflict.pl - Mailing list pgsql-hackers

From Ayush Tiwari
Subject Re: Stabilize recovery conflict stats checks in 031_recovery_conflict.pl
Date
Msg-id CAJTYsWUNRGZBy+cvXRN4srzm98kfjM7W_HVJbFRzryABp-Fuew@mail.gmail.com
Whole thread
In response to Re: Stabilize recovery conflict stats checks in 031_recovery_conflict.pl  (Alexander Lakhin <exclusion@gmail.com>)
Responses Re: [PATCH v1 0/7] Wait event timing and tracing instrumentation
List pgsql-hackers
Hi,

On Fri, 4 Sept 2026 at 09:30, Alexander Lakhin <exclusion@gmail.com> wrote:
>
> Hello Ayush and Fujii-san,
>
> 04.09.2026 06:07, Fujii Masao wrote:
> > On Fri, Sep 4, 2026 at 6:48 AM Ayush Tiwari <ayushtiwari.slg01@gmail.com> wrote:
> >> Thoughts?
> > Thanks for the patches! They look good to me.
>
> Thank you for paying attention to my reports!
>
> I'm not sure about the autovacuum fix -- I've found that I tried the same
> targeted autovacuum = false as Melanie proposed back then ([1]) and the
> test still failed. Please let me recheck carefully if it's still an issue.
>
> Regarding v1-0001, I've tested it with the repro.patch applied and
> confirmed it works. Thank you!
>
> [1] https://www.postgresql.org/message-id/714729ea-f536-0faf-353b-cbd5c8f241ea%40gmail.com

Thanks for testing and confirming 0001.

One detail I noticed in [1] is that the posted change sets
autovacuum_enabled=false on the initial table. That table is later dropped
and recreated for the recovery deadlock test, so the reloption does not
carry over. 0002 sets it on the recreated table instead.

However, if you tested that placement as well, there may be another race.

Regards,
Ayush
Attachment

pgsql-hackers by date:

Previous
From: Paul A Jungwirth
Date:
Subject: FOR PORTION OF elog on non-polymorphic opcintype
Next
From: Chao Li
Date:
Subject: Re: Rename of varatt_external to varatt_external_oid