Move replication slot release to before_shmem_exit().
authorAndres Freund <andres@anarazel.de>
Tue, 15 Feb 2022 00:44:28 +0000 (16:44 -0800)
committerAndres Freund <andres@anarazel.de>
Tue, 15 Feb 2022 01:08:17 +0000 (17:08 -0800)
commit2f6501fa3c54bbe4568e3bcccd9a60d26a46b5ee
treee1ced75463fc0cbb0de404d27d6227ecef1e5663
parentb45fa793406d6007a787c586c1960b1ffc2ef2fb
Move replication slot release to before_shmem_exit().

Previously, replication slots were released in ProcKill() on error, resulting
in reporting replication slot drop of ephemeral slots after the stats
subsystem was already shut down.

To fix this problem, move replication slot release to a before_shmem_exit()
hook that is called before the stats collector shuts down. There wasn't really
a good reason for the slot handling to be in ProcKill() anyway.

Patch by Masahiko Sawada, with very minor polishing by me.

I, Andres, wrote a test for dropping slots during process exit, but there may
be some OS dependent issues around the number of times FATAL error messages
are displayed due to a still debated libpq issue. So that test will be
committed separately / later.

Reviewed-By: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Reviewed-By: Andres Freund <andres@anarazel.de>
Author: Masahiko Sawada <sawada.mshk@gmail.com>
Discussion: https://postgr.es/m/CAD21AoDAeEpAbZEyYJsPZJUmSPaRicVSBObaL7sPaofnKz+9zg@mail.gmail.com
src/backend/replication/slot.c
src/backend/storage/lmgr/proc.c
src/backend/tcop/postgres.c
src/backend/utils/init/postinit.c
src/include/replication/slot.h