Re: XLog size reductions: Reduced XLog record header size for PG17
Matthias van de Meent <boekewurm+postgres@gmail.com>
From: Matthias van de Meent <boekewurm+postgres@gmail.com>
To: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Cc: Heikki Linnakangas <hlinnaka@iki.fi>, Robert Haas <robertmhaas@gmail.com>, Dilip Kumar <dilipbalaut@gmail.com>, Andres Freund <andres@anarazel.de>
Date: 2023-06-30T15:36:40Z
Lists: pgsql-hackers
Attachments
- v2-0004-Move-rmgr-info-bits-into-their-own-field-in-the-x.patch (application/octet-stream) patch v2-0004
- v2-0003-Rename-rmgr_identify-info-bits-argument-to-rmgrin.patch (application/octet-stream) patch v2-0003
- v2-0001-Reduce-overhead-of-small-block-data-in-xlog-recor.patch (application/octet-stream) patch v2-0001
- v2-0002-Replace-accesses-to-rmgr-s-XLogRecord-info-bit-ac.patch (application/octet-stream) patch v2-0002
- v2-0005-Stop-the-logging-of-XIDs-in-records-where-the-XID.patch (application/octet-stream) patch v2-0005
- v2-0006-Reformat-the-XLog-record-header.patch (application/octet-stream) patch v2-0006
Hi, The attached v2 patchset contains some small fixes for the failing cfbot 32-bit tests - at least locally it does so. I'd overlooked one remaining use of MAXALIGN64() in xlog.c in the last patch of the set, which has now been updated to XLP_ALIGN as well. Additionally, XLP_ALIGN has been updated to use TYPEALIGN64 instead of TYPEALIGN so that we don't lose bits of the aligned value in 32-bit systems. Kind regards, Matthias van de Meent Neon (https://neon.tech/)