Hi, On Mon, Apr 27, 2026 at 8:59 PM shveta malik <[email protected]> wrote: > > > I initially considered an injection point but chose polling since the > > TAP test reproduced the bug consistently with hundreds of tables on my > > dev system. I've now added an injection point for predictability. > > > > I adjusted the commit message a bit. Please find the attached v3 patch > > for further review. Thank you! > > > > Thanks Bharath. I have just one minor comment: > > + INJECTION_POINT("pg-get-publication-tables-build-list", NULL); > > Shall we name it as 'pg-get-publication-tables-list-built' to be more > meaningful, as we are pausing after list is built.
Sure. How about pg-get-publication-tables-after-list-built? It's 42 bytes, under the INJ_NAME_MAXLEN of 64 bytes, but meaningful. -- Bharath Rupireddy Amazon Web Services: https://aws.amazon.com
