pgsql: Reuse zstd decompression contexts when restoring WAL images - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Reuse zstd decompression contexts when restoring WAL images
Date
Msg-id E1xEcKL-00000000kf5-04ht@gemulon.postgresql.org
Whole thread
List pgsql-committers
Reuse zstd decompression contexts when restoring WAL images

Restoring each zstd-compressed full-page image currently creates and
destroys a decompression context.  One decompression context gets now
saved in a XLogReaderState on first use, reused for the decompression of
any follow-up zstd-compressed pages.

The decompression context is freed in XLogReaderFree().  In non-FRONTEND
builds, a memory context reset callback is registered to make sure that
the decompression context does not leak.  Note that zstd has APIs to
either initialize a pre-allocated decompression context or assign custom
allocation and free routines, but none of these are available for
builds that dynamically link to the zstd library (ZSTDLIB_STATIC_API).

Tomas Vondra has done some benchmarking showing that for a FPI-heavy set
of WAL records, reusing the decompression context can nicely improve the
recovery time, bringing it close to lz4.  In my own measurements, I have
seen up to a 4% faster recovery time when replaying records made mostly
of zstd-compressed full-page images between HEAD and this improvement.
Andrey has seen better numbers than that.

Author: Andrey Borodin <x4mmm@yandex-team.ru>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://www.postgresql.org/message-id/asRsOl73nf8GeBFd@paquier.xyz

Branch
------
master

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

Modified Files
--------------
src/backend/access/transam/xlogreader.c | 51 +++++++++++++++++++++++++++++++--
src/include/access/xlogreader.h         |  9 ++++++
2 files changed, 57 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Masahiko Sawada
Date:
Subject: pgsql: Fix slotsync when logical decoding is disabled and re-enabled.
Next
From: Amit Langote
Date:
Subject: pgsql: Refuse RI fast-path row locks in read-only transactions