Re: Background writer and checkpointer in crash recovery

Robert Haas <robertmhaas@gmail.com>

View thread View on postgr.es

From: Robert Haas <robertmhaas@gmail.com>

To: Thomas Munro <thomas.munro@gmail.com>

Cc: Aleksander Alekseev <aleksander@timescale.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2021-08-02T13:17:29Z

Lists: pgsql-hackers

Attachments

0001-Further-simplify-a-bit-of-logic-in-StartupXLOG.patch (application/octet-stream) patch 0001

On Mon, Aug 2, 2021 at 1:37 AM Thomas Munro <thomas.munro@gmail.com> wrote:
> I pushed 0001.

That's great. I just realized that this leaves us with identical
RequestCheckpoint() calls in two nearby places. Is there any reason
not to further simplify as in the attached?

-- 
Robert Haas
EDB: http://www.enterprisedb.com

Commits

Clear ps display of startup process at the end of recovery
- 848c323c1295 15.0 landed
- fbb5f54b67c2 16.0 landed
Further simplify a bit of logic in StartupXLOG().
- 8f7c8e2bef2b 15.0 landed
Run checkpointer and bgwriter in crash recovery.
- 7ff23c6d277d 15.0 landed
Add some checkpoint/restartpoint status to ps display
- df9274adf309 14.0 cited
Start background writer during archive recovery. Background writer now performs
- cdd46c765488 8.4.0 cited