From: | Jeff Davis <jdavis(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: pg_walinspect: fix case where flush LSN is in the middle of a re |
Date: | 2022-04-30 16:21:33 |
Message-ID: | E1nkpqf-001tOJ-EQ@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
pg_walinspect: fix case where flush LSN is in the middle of a record.
Instability in the test for pg_walinspect revealed that
pg_get_wal_records_info_till_end_of_wal(x) would try to decode all the
records with a start LSN earlier than the flush LSN, even though that
might include a partial record at the end of the range. In that case,
read_local_xlog_page_no_wait() would return NULL when it tried to read
past the flush LSN, which would be interpreted as an error by the
caller. That caused a test failure only on a BF animal that had been
restarted recently, but could be expected to happen in the wild quite
easily depending on the alignment of various parameters.
Fix by using private data in read_local_xlog_page_no_wait() to signal
end-of-wal to the caller, so that it can be properly distinguished
from a real error.
Discussion: https://postgr.es/m/Ymd/e5eeZMNAkrXo%40paquier.xyz
Discussion: https://postgr.es/m/111657.1650910309@sss.pgh.pa.us
Authors: Thomas Munro, Bharath Rupireddy.
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/ed57cac84d1c5642737dab1e4c4b8cb4f0c4305f
Modified Files
--------------
contrib/pg_walinspect/pg_walinspect.c | 59 +++++++++++++++++++---------------
src/backend/access/transam/xlogutils.c | 11 +++++++
src/include/access/xlogutils.h | 6 ++++
3 files changed, 50 insertions(+), 26 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2022-04-30 16:24:02 | Re: pgsql: Add contrib/pg_walinspect. |
Previous Message | Tom Lane | 2022-04-30 15:54:34 | pgsql: Tighten enforcement of variable CONSTANT markings in plpgsql. |