pgsql: Replace XLOG_INCLUDE_XID flag with a more localized flag. - Mailing list pgsql-committers

From Amit Kapila
Subject pgsql: Replace XLOG_INCLUDE_XID flag with a more localized flag.
Date
Msg-id E1mhkF6-00042g-JT@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Replace XLOG_INCLUDE_XID flag with a more localized flag.

Commit 0bead9af484c introduced XLOG_INCLUDE_XID flag to indicate that the
WAL record contains subXID-to-topXID association. It uses that flag later
to mark in CurrentTransactionState that top-xid is logged so that we
should not try to log it again with the next WAL record in the current
subtransaction. However, we can use a localized variable to pass that
information.

In passing, change the related function and variable names to make them
consistent with what the code is actually doing.

Author: Dilip Kumar
Reviewed-by: Alvaro Herrera, Amit Kapila
Discussion: https://postgr.es/m/E1mSoYz-0007Fh-D9@gemulon.postgresql.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/71db6459e6e4ef623e98f3b1e3e9fed1bfb0ae3b

Modified Files
--------------
src/backend/access/transam/xact.c       | 103 ++++++++++++++++----------------
src/backend/access/transam/xlog.c       |  13 +++-
src/backend/access/transam/xloginsert.c |  26 ++++----
src/include/access/xact.h               |   4 +-
src/include/access/xlog.h               |   4 +-
5 files changed, 82 insertions(+), 68 deletions(-)


pgsql-committers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: pgsql: Replace unicode characters in comments with ascii
Next
From: Amit Kapila
Date:
Subject: pgsql: Move MarkCurrentTransactionIdLoggedIfAny() out of the critical s