diff options
| author | Andres Freund <andres@anarazel.de> | 2022-01-21 11:22:55 -0800 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2022-01-21 11:24:12 -0800 |
| commit | 2b7dbe4bd5872ae5c6780e1a4121c2a5a8401e7c (patch) | |
| tree | 90c96046dbb6aba907ef4654c80b2dcac75165d7 /src | |
| parent | 700e8fe6d414ebaf9820fef364097d9b83cae7fd (diff) | |
| download | postgresql-2b7dbe4bd5872ae5c6780e1a4121c2a5a8401e7c.tar.gz postgresql-2b7dbe4bd5872ae5c6780e1a4121c2a5a8401e7c.zip | |
fsync pg_logical/mappings in CheckPointLogicalRewriteHeap().
While individual logical rewrite files were synced to disk, the directory was
not. On some filesystems that could lead to loosing directory entries after a
crash.
Reported-By: Tom Lane <tgl@sss.pgh.pa.us>
Author: Nathan Bossart <bossartn@amazon.com>
Discussion: https://postgr.es/m/867F2E29-2782-4869-970E-B984C6D35A8F@amazon.com
Backpatch: 10-
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/heap/rewriteheap.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/access/heap/rewriteheap.c b/src/backend/access/heap/rewriteheap.c index 1aff62cd423..15bef9f7d30 100644 --- a/src/backend/access/heap/rewriteheap.c +++ b/src/backend/access/heap/rewriteheap.c @@ -1289,4 +1289,7 @@ CheckPointLogicalRewriteHeap(void) } } FreeDir(mappings_dir); + + /* persist directory entries to disk */ + fsync_fname("pg_logical/mappings", true); } |