pgsql: Remove XLogFileInit() ability to unlink a pre-existing file. - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Remove XLogFileInit() ability to unlink a pre-existing file.
Date
Msg-id E1u1JBR-002v6b-05@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove XLogFileInit() ability to unlink a pre-existing file.

Only initdb used it.  initdb refuses to operate on a non-empty directory
and generally does not cope with pre-existing files of other kinds.
Hence, use the opportunity to simplify.

This commit has been applied as of 421484f79c0b in v15 and newer
versions.  This is required on stable branches of v13 and v14 to fix a
regression reported by Noah Misch, introduced by 1f95181b44c8, causing
spurious failures in archive recovery (neither streaming nor archive
recovery) with concurrent restartpoints.  The backpatched versions of
the patches have been aligned on these branches by me, Noah Misch is the
author.  Tests have been conducted by the both of us.

Reported-by: Arun Thirupathi
Author: Noah Misch <noah@leadboat.com>
Discussion: https://postgr.es/m/20210202151416.GB3304930@rfd.leadboat.com
Discussion: https://postgr.es/m/20250306193013.36.nmisch@google.com
Backpatch-through: 13

Branch
------
REL_14_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/b494640e8297db8b626d8ffa775cb491f7d2e7e2
Author: Noah Misch <noah@leadboat.com>

Modified Files
--------------
src/backend/access/transam/xlog.c     | 61 ++++++++++++++---------------------
src/backend/replication/walreceiver.c |  4 +--
src/include/access/xlog.h             |  2 +-
3 files changed, 28 insertions(+), 39 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: In XLogFileInit(), fix *use_existent postcondition to suit calle
Next
From: Michael Paquier
Date:
Subject: pgsql: Revert "Add HINT for restartpoint race with KeepFileRestoredFrom