pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix instability with WAL fsync test in stats.sql
Date: Sat, 12 Apr 2025 04:24:08 +0000
Message-ID: <E1u3SPY-003qab-0d@gemulon.postgresql.org> (raw)

Fix instability with WAL fsync test in stats.sql

A backend using wal_sync_method set to "open_sync" or "open_datasync"
would fail the test checking the WAL sync data in pg_stat_io.  These
modes guarantee that a sync is done when WAL is written to disk, and the
data checked by the test is not incremented in this case,
issue_xlog_fsync() doing nothing.

Oversight in commit a051e71e28a1.

Author: Sami Imseih <samimseih@gmail.com>
Discussion: https://postgr.es/m/CAA5RZ0uxwg3xAi4nvdBMJ-zJQEeyg+RotuU+ebM2F6CKmnvaYA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/fdb69dd582427e4976897d544867e65a0e767002

Modified Files
--------------
src/test/regress/expected/stats.out | 1 +
src/test/regress/sql/stats.sql      | 1 +
2 files changed, 2 insertions(+)



Message-ID: <E1u3SPY-003qab-0d@gemulon.postgresql.org>
Permalink:  ../E1u3SPY-003qab-0d@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1u3SPY-003qab-0d@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: michael@paquier.xyz, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix instability with WAL fsync test in stats.sql
  In-Reply-To: <E1u3SPY-003qab-0d@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox