On Mon, Feb 26, 2024 at 02:10:49PM +0500, Andrey M. Borodin wrote: > So that we could do something like > > ok(node_standby->await_injection_point(“CreateRestartPoint”,”checkpointer"));
It would be more flexible with a full string to describe the test rather than a process name in the second argument. > IMO, this could make many tests cleaner. > Or, perhaps, it’s a functionality for a future development? This could just happen as separate refactoring, I guess. But I'd wait to see if more tests requiring scans to pg_stat_activity pop up. For example, the test just posted here does not rely on that: https://www.postgresql.org/message-id/zdyzya4yrnapw...@ip-10-97-1-34.eu-west-3.compute.internal -- Michael
signature.asc
Description: PGP signature
