Fix correctness issue with computation of FPI size in WAL stats
authorMichael Paquier <michael@paquier.xyz>
Wed, 29 Oct 2025 00:13:31 +0000 (09:13 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 29 Oct 2025 00:13:31 +0000 (09:13 +0900)
commitd3111cb753e81a64c1a4417ed6de98a90a04432a
tree364cd382d129ce691a3e2914e06e4b2b87ed39a0
parentb3ce55f413cdf70b1bc4724052fb4eacf9de239a
Fix correctness issue with computation of FPI size in WAL stats

XLogRecordAssemble() may be called multiple times before inserting a
record in XLogInsertRecord(), and the amount of FPIs generated inside
a record whose insertion is attempted multiple times may vary.

The logic added in f9a09aa29520 touched directly pgWalUsage in
XLogRecordAssemble(), meaning that it could be possible for pgWalUsage
to be incremented multiple times for a single record.  This commit
changes the code to use the same logic as the number of FPIs added to a
record, where XLogRecordAssemble() returns this information and feeds it
to XLogInsertRecord(), updating pgWalUsage only when a record is
inserted.

Reported-by: Shinya Kato <shinya11.kato@gmail.com>
Discussion: https://postgr.es/m/CAOzEurSiSr+rusd0GzVy8Bt30QwLTK=ugVMnF6=5WhsSrukvvw@mail.gmail.com
src/backend/access/transam/xlog.c
src/backend/access/transam/xloginsert.c
src/include/access/xlog.h