diff options
| author | Michael Paquier <michael@paquier.xyz> | 2024-03-07 09:57:52 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2024-03-07 09:57:52 +0900 |
| commit | 65db0cfb4c036b14520a22dba5a858185b713643 (patch) | |
| tree | 6551f001640c37f1523cd235f25fac3345327864 /src/backend | |
| parent | 099ca50bd41c1a72fe5a086bb095858ea74c88de (diff) | |
| download | postgresql-65db0cfb4c036b14520a22dba5a858185b713643.tar.gz postgresql-65db0cfb4c036b14520a22dba5a858185b713643.zip | |
Revert "Add recovery TAP test for race condition with slot invalidations"
This reverts commit 08a52ab151ca, due to some sporadic instability in
the test. Getting the test right should require some redesign with a
second injection point, but let's revert it for now to avoid these
issues in the CI as a lot of patches are under discussion in this last
commit fest.
Per buildfarm members hachi and gokiburi.
Discussion: https://postgr.es/m/ZekQQHCrIqLVpGz5@paquier.xyz
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/replication/slot.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/src/backend/replication/slot.c b/src/backend/replication/slot.c index 02ae27499b5..2614f98ddd2 100644 --- a/src/backend/replication/slot.c +++ b/src/backend/replication/slot.c @@ -53,7 +53,6 @@ #include "storage/proc.h" #include "storage/procarray.h" #include "utils/builtins.h" -#include "utils/injection_point.h" /* * Replication slot on-disk data structure. @@ -1659,14 +1658,6 @@ InvalidatePossiblyObsoleteSlot(ReplicationSlotInvalidationCause cause, last_signaled_pid = active_pid; terminated = true; conflict_prev = conflict; - - /* - * This injection point needs to be after kill() to ensure - * that the slot is not "active" anymore. It also has to be - * after ReportSlotInvalidation() to ensure that the - * invalidation message is logged. - */ - INJECTION_POINT("terminate-process-holding-slot"); } /* Wait until the slot is released. */ |