diff options
| author | Andres Freund <andres@anarazel.de> | 2022-07-05 11:01:10 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2022-07-05 11:01:10 -0700 |
| commit | 3f8148c256e067dc2e8929ed174671ba7dc3339c (patch) | |
| tree | 26d7ecf745164550eed9d462cc56955545ff9c18 /src/backend/utils/init/postinit.c | |
| parent | b9eb0ff09eb843566645679c3ab65b3c9d12c08b (diff) | |
| download | postgresql-3f8148c256e067dc2e8929ed174671ba7dc3339c.tar.gz postgresql-3f8148c256e067dc2e8929ed174671ba7dc3339c.zip | |
Revert 019_replslot_limit.pl related debugging aids.
This reverts most of 91c0570a791, f28bf667f60, fe0972ee5e6, afdeff10526. The
only thing left is the retry loop in 019_replslot_limit.pl that avoids
spurious failures by retrying a couple times.
We haven't seen any hard evidence that this is caused by anything but slow
process shutdown. We did not find any cases where walsenders did not vanish
after waiting for longer. Therefore there's no reason for this debugging code
to remain.
Discussion: https://postgr.es/m/20220530190155.47wr3x2prdwyciah@alap3.anarazel.de
Backpatch: 15-
Diffstat (limited to 'src/backend/utils/init/postinit.c')
| -rw-r--r-- | src/backend/utils/init/postinit.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c index 6b9082604fb..a5c208a20a8 100644 --- a/src/backend/utils/init/postinit.c +++ b/src/backend/utils/init/postinit.c @@ -1231,24 +1231,6 @@ ShutdownPostgres(int code, Datum arg) * them explicitly. */ LockReleaseAll(USER_LOCKMETHOD, true); - - /* - * temp debugging aid to analyze 019_replslot_limit failures - * - * If an error were thrown outside of a transaction nothing up to now - * would have released lwlocks. We probably will add an - * LWLockReleaseAll(). But for now make it easier to understand such cases - * by warning if any lwlocks are held. - */ -#ifdef USE_ASSERT_CHECKING - { - int held_lwlocks = LWLockHeldCount(); - - if (held_lwlocks) - elog(WARNING, "holding %d lwlocks at the end of ShutdownPostgres()", - held_lwlocks); - } -#endif } |