Re: [PATCH] Table sync race with REFRESH PUBLICATION - Mailing list pgsql-hackers

From Peter Smith
Subject Re: [PATCH] Table sync race with REFRESH PUBLICATION
Date
Msg-id CAHut+Pv2FiR-rtoMxAxPJ6kQcoYiFWmPoS6n2ptHz4d_pkc83g@mail.gmail.com
Whole thread
In response to Re: [PATCH] Table sync race with REFRESH PUBLICATION  (Ayush Tiwari <ayushtiwari.slg01@gmail.com>)
List pgsql-hackers
> Attached is v2 with those changes. Thoughts?
>

Hi Ayush.

//////////
Patch v2-0001
//////////

Patch 0001 looks good to me, but more eyes than mine need to review it.

//////////
Patch v2-0002
//////////

I confirmed the test fails when patch 0001 is not applied and passes
when it is applied.

I assume the test code is AI-generated, because it looks way less
human-readable than a typical TAP test. If the test is ever intended
to be pushed, then it needs lots of comments to describe what it is
doing and expecting at each step.

FWIW, I've attached a copy of that test code augmented with some
AI-generated comments. It's still very difficult -- I don't claim to
understand everything it is doing -- but the comments might be helpful
to somebody. PSA.

======
Kind Regards,
Peter Smith.
Fujitsu Australia

Attachment

pgsql-hackers by date:

Previous
From: Chao Li
Date:
Subject: Re: pg_resetwal with replication slot (17.11)
Next
From: Chao Li
Date:
Subject: Re: Parallel vacuum: I/O timings in the log leave out the parallel workers