diff options
| author | Noah Misch <noah@leadboat.com> | 2024-02-01 13:44:19 -0800 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2024-02-01 13:44:23 -0800 |
| commit | 8fa4a1ac61189efffb8b851ee77e1bc87360c445 (patch) | |
| tree | 0d6cf5c6495cd17aedd0d2e9e31bcededd740ad6 /src/backend/commands/dbcommands.c | |
| parent | d493bed28f7f6c77051bba3dde383e0ff78d3a19 (diff) | |
| download | postgresql-8fa4a1ac61189efffb8b851ee77e1bc87360c445.tar.gz postgresql-8fa4a1ac61189efffb8b851ee77e1bc87360c445.zip | |
Sync PG_VERSION file in CREATE DATABASE.
An OS crash could leave PG_VERSION empty or missing. The same symptom
appeared in a backup by block device snapshot, taken after the next
checkpoint and before the OS flushes the PG_VERSION blocks. Device
snapshots are not a documented backup method, however. Back-patch to
v15, where commit 9c08aea6a3090a396be334cc58c511edab05776a introduced
STRATEGY=WAL_LOG and made it the default.
Discussion: https://postgr.es/m/20240130195003.0a.nmisch@google.com
Diffstat (limited to 'src/backend/commands/dbcommands.c')
| -rw-r--r-- | src/backend/commands/dbcommands.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/backend/commands/dbcommands.c b/src/backend/commands/dbcommands.c index 79c59889633..5ced6da20b6 100644 --- a/src/backend/commands/dbcommands.c +++ b/src/backend/commands/dbcommands.c @@ -507,6 +507,14 @@ CreateDirAndVersionFile(char *dbpath, Oid dbid, Oid tsid, bool isRedo) } pgstat_report_wait_end(); + pgstat_report_wait_start(WAIT_EVENT_VERSION_FILE_SYNC); + if (pg_fsync(fd) != 0) + ereport(data_sync_elevel(ERROR), + (errcode_for_file_access(), + errmsg("could not fsync file \"%s\": %m", versionfile))); + fsync_fname(dbpath, true); + pgstat_report_wait_end(); + /* Close the version file. */ CloseTransientFile(fd); |