pgsql: Unify two isLogSwitch tests in XLogInsertRecord.

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Unify two isLogSwitch tests in XLogInsertRecord.
Date: 2023-10-12 18:09:56
Message-ID: E1qr08B-000Dqw-Lr@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Unify two isLogSwitch tests in XLogInsertRecord.

An upcoming patch wants to introduce an additional special case in
this function. To keep that as cheap as possible, minimize the amount
of branching that we do based on whether this is an XLOG_SWITCH
record.

Additionally, and also in the interest of keeping the overhead of
special-case code paths as low as possible, apply likely() to the
non-XLOG_SWITCH case, since only a very tiny fraction of WAL records
will be XLOG_SWITCH records.

Patch by me, reviewed by Dilip Kumar, Amit Kapila, Andres Freund,
and Michael Paquier.

Discussion: http://postgr.es/m/CA+TgmoYy-Vc6G9QKcAKNksCa29cv__czr+N9X_QCxEfQVpp_8w@mail.gmail.com

Branch
------
master

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

Modified Files
--------------
src/backend/access/transam/xlog.c | 101 ++++++++++++++++++++++----------------
1 file changed, 58 insertions(+), 43 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2023-10-13 02:00:18 pgsql: doc: Mention timezone(zone, time) in section for AT TIME ZONE
Previous Message David Rowley 2023-10-12 12:15:30 pgsql: Fix runtime partition pruning for HASH partitioned tables