pgsql: Ensure cleanup of orphan archive status files - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Ensure cleanup of orphan archive status files
Date
Msg-id E1gWEeD-0006Zm-FU@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Ensure cleanup of orphan archive status files

When a WAL segment is recycled, its ".ready" and ".done" status files
get also automatically removed, however this is not done in a durable
manner.  Hence, in a subsequent crash, it could be possible that a
".ready" status file is still around with its corresponding segment
already gone.

If the backend reaches such a state, the archive command would most
likely complain about a segment non-existing and would keep retrying,
causing WAL segments to bloat pg_wal/, potentially making Postgres crash
hard when running out of space.

As status files are removed after each individual segment, using
durable_unlink() does not completely close the window either, as a crash
could happen between the moment the WAL segment is recycled and the
moment its status files are removed.  This has also some performance
impact with the additional fsync() calls needed to make the removal in a
durable manner.  Doing the cleanup at recovery is not cost-free either
as this makes crash recovery potentially take longer than necessary.

So, instead, as per an idea of Stephen Frost, make the archiver aware of
orphan status files and remove them on-the-fly if the corresponding
segment goes missing.  Removal failures follow a model close to what
happens for WAL segments, where multiple attempts are done before giving
up temporarily, and where a successful orphan removal makes the archiver
move immediately to the next WAL segment thought as ready to be
archived.

Author: Michael Paquier
Reviewed-by: Nathan Bossart, Andres Freund, Stephen Frost, Kyotaro
Horiguchi
Discussion: https://postgr.es/m/20180928032827.GF1500@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/6d8727f95e7043d52ea8f6d62f7bcf81fdeaa658

Modified Files
--------------
src/backend/postmaster/pgarch.c | 55 +++++++++++++++++++++++++++++++++++++++++
1 file changed, 55 insertions(+)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Add timestamp of last received message from standby topg_stat_r
Next
From: Stephen Frost
Date:
Subject: pgsql: Remove dead code in toast_fetch_datum_slice