From: | Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Remove unnecessary smgrimmedsync() when creating unlogged table. |
Date: | 2023-09-15 14:53:25 |
Message-ID: | E1qhACD-003zTH-8J@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Remove unnecessary smgrimmedsync() when creating unlogged table.
This became safe after commit 4b4798e138. The smgrcreate() call will
now register the segment for syncing at the next checkpoint, so we
don't need to sync it here. If a checkpoint happens before the
creation is WAL-logged, the records will be replayed when starting
recovery from the checkpoint. If a checkpoint happens after the WAL
logging, the checkpoint will fsync() it.
In the passing, clarify a comment in smgrDoPendingSyncs().
Discussion: https://www.postgresql.org/message-id/6e5bbc08-cdfc-b2b3-9e23-1a914b9850a9%40iki.fi
Reviewed-by: Robert Haas
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/18724af9e83bb9123e0e1cd09751aef4ba30039e
Modified Files
--------------
src/backend/access/heap/heapam_handler.c | 10 +++-------
src/backend/catalog/storage.c | 2 +-
2 files changed, 4 insertions(+), 8 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2023-09-15 17:09:45 | pgsql: Rename variable for code clarity |
Previous Message | Daniel Gustafsson | 2023-09-14 09:21:03 | pgsql: Quote filenames in error messages |