pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Robert Haas <rhaas@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Prevent summarizer hang when summarize_wal turned off and back o
Date: Fri, 28 Jun 2024 12:45:51 +0000
Message-ID: <E1sNAz9-003Mkl-0b@gemulon.postgresql.org> (raw)

Prevent summarizer hang when summarize_wal turned off and back on.

Before this commit, when the WAL summarizer started up or recovered
from an error, it would resume summarization from wherever it left
off. That was OK normally, but wrong if summarize_wal=off had been
turned off temporary, allowing some WAL to be removed, and then turned
back on again. In such cases, the WAL summarizer would simply hang
forever. This commit changes the reinitialization sequence for WAL
summarizer to rederive the starting position in the way we were
already doing at initial startup, fixing the problem.

Per report from Israel Barth Rubio. Reviewed by Tom Lane.

Discussion: http://postgr.es/m/CA+TgmoYN6x=YS+FoFOS6=nr6=qkXZFWhdiL7k0oatGwug2hcuA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/065583cf460f980a182498941ac52810f709a897

Modified Files
--------------
src/backend/access/transam/xlog.c      |  2 +-
src/backend/postmaster/walsummarizer.c | 83 ++++++++++++++++++----------------
src/include/postmaster/walsummarizer.h |  3 +-
3 files changed, 45 insertions(+), 43 deletions(-)



Message-ID: <E1sNAz9-003Mkl-0b@gemulon.postgresql.org>
Permalink:  ../E1sNAz9-003Mkl-0b@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1sNAz9-003Mkl-0b@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: rhaas@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Prevent summarizer hang when summarize_wal turned off and back o
  In-Reply-To: <E1sNAz9-003Mkl-0b@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