pgsql: Move check for fsync=off so that pendingOps still gets cleared. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Move check for fsync=off so that pendingOps still gets cleared.
Date
Msg-id E1jZ6cL-000072-Vk@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Move check for fsync=off so that pendingOps still gets cleared.

Commit 3eb77eba5a moved the loop and refactored it, and inadvertently
changed the effect of fsync=off so that it also skipped removing entries
from the pendingOps table. That was not intentional, and leads to an
assertion failure if you turn fsync on while the server is running and
reload the config.

Backpatch-through: 12-
Reviewed-By: Thomas Munro
Discussion: https://www.postgresql.org/message-id/3cbc7f4b-a5fa-56e9-9591-c886deb07513%40iki.fi

Branch
------
REL_12_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/891a2007e3987545a2543c1f04cd0682576b39fa

Modified Files
--------------
src/backend/storage/sync/sync.c | 148 ++++++++++++++++++++--------------------
1 file changed, 75 insertions(+), 73 deletions(-)


pgsql-committers by date:

Previous
From: Amit Kapila
Date:
Subject: pgsql: Fix the MSVC build for versions 2015 and later.
Next
From: Alexander Korotkov
Date:
Subject: pgsql: Fix amcheck for page checks concurrent to replay of btree page d