diff options
| author | Alexander Korotkov <akorotkov@postgresql.org> | 2025-08-22 18:44:39 +0300 |
|---|---|---|
| committer | Alexander Korotkov <akorotkov@postgresql.org> | 2025-08-22 19:26:38 +0300 |
| commit | c13070a27b63d9ce4850d88a63bf889a6fde26f0 (patch) | |
| tree | 11fcb0a227de116687a253897150f857f2693bf2 /src/bin/scripts/scripts_parallel.c | |
| parent | b63952a7811319a6525a9ea23afa294a900fa895 (diff) | |
Revert "Get rid of WALBufMappingLock"HEADorigin/masterorigin/HEADmaster
This reverts commit bc22dc0e0ddc2dcb6043a732415019cc6b6bf683.
It appears that conditional variables are not suitable for use inside
critical sections. If WaitLatch()/WaitEventSetWaitBlock() face postmaster
death, they exit, releasing all locks instead of PANIC. In certain
situations, this leads to data corruption.
Reported-by: Andrey Borodin <x4mmm@yandex-team.ru>
Discussion: https://postgr.es/m/B3C69B86-7F82-4111-B97F-0005497BB745%40yandex-team.ru
Reviewed-by: Andrey Borodin <x4mmm@yandex-team.ru>
Reviewed-by: Aleksander Alekseev <aleksander@tigerdata.com>
Reviewed-by: Kirill Reshke <reshkekirill@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Reviewed-by: Thomas Munro <thomas.munro@gmail.com>
Reviewed-by: Tomas Vondra <tomas@vondra.me>
Reviewed-by: Andres Freund <andres@anarazel.de>
Reviewed-by: Yura Sokolov <y.sokolov@postgrespro.ru>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Backpatch-through: 18
Diffstat (limited to 'src/bin/scripts/scripts_parallel.c')
0 files changed, 0 insertions, 0 deletions