Re: [PATCH] Table sync race with REFRESH PUBLICATION

From: Peter Smith <smithpb2250(at)gmail(dot)com>
To: Ayush Tiwari <ayushtiwari(dot)slg01(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Table sync race with REFRESH PUBLICATION
Date: 2026-09-29 01:59:35
Message-ID: CAHut+Pv2FiR-rtoMxAxPJ6kQcoYiFWmPoS6n2ptHz4d_pkc83g@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: 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 Content-Type Size
v2-0002-tests-with-comments.nocfbot application/octet-stream 11.4 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Previous Message Chao Li 2026-09-29 01:12:22 Re: pg_resetwal with replication slot (17.11)