pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
pgsql: Merge copies of converting an XID to a FullTransactionId.
2+ messages / 1 participants
[nested] [flat]

* pgsql: Merge copies of converting an XID to a FullTransactionId.
@ 2025-01-25 19:29  Noah Misch <noah@leadboat.com>
  0 siblings, 0 replies; 2+ messages in thread

From: Noah Misch @ 2025-01-25 19:29 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

Merge copies of converting an XID to a FullTransactionId.

Assume twophase.c is the performance-sensitive caller, and preserve its
choice of unlikely() branch hint.  Add some retrospective rationale for
that choice.  Back-patch to v17, for the next commit to use it.

Reviewed (in earlier versions) by Michael Paquier.

Discussion: https://postgr.es/m/17821-dd8c334263399284@postgresql.org
Discussion: https://postgr.es/m/20250116010051.f3.nmisch@google.com

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/1587f7b9fc1a16df8b01b4f5f9e3c949325160b2

Modified Files
--------------
contrib/amcheck/verify_heapam.c         |  4 +-
src/backend/access/transam/twophase.c   | 26 +++--------
src/backend/access/transam/xlogreader.c | 18 +-------
src/backend/utils/adt/xid8funcs.c       | 79 ++++++++++-----------------------
src/include/access/transam.h            | 43 ++++++++++++++++++
5 files changed, 77 insertions(+), 93 deletions(-)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* pgsql: Merge copies of converting an XID to a FullTransactionId.
@ 2025-01-25 19:29  Noah Misch <noah@leadboat.com>
  0 siblings, 0 replies; 2+ messages in thread

From: Noah Misch @ 2025-01-25 19:29 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

Merge copies of converting an XID to a FullTransactionId.

Assume twophase.c is the performance-sensitive caller, and preserve its
choice of unlikely() branch hint.  Add some retrospective rationale for
that choice.  Back-patch to v17, for the next commit to use it.

Reviewed (in earlier versions) by Michael Paquier.

Discussion: https://postgr.es/m/17821-dd8c334263399284@postgresql.org
Discussion: https://postgr.es/m/20250116010051.f3.nmisch@google.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/81772a495ec98d36eabf5cc294e7031a9545c5c1

Modified Files
--------------
contrib/amcheck/verify_heapam.c         |  4 +-
src/backend/access/transam/twophase.c   | 26 +++--------
src/backend/access/transam/xlogreader.c | 18 +-------
src/backend/utils/adt/xid8funcs.c       | 79 ++++++++++-----------------------
src/include/access/transam.h            | 43 ++++++++++++++++++
5 files changed, 77 insertions(+), 93 deletions(-)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2025-01-25 19:29 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2025-01-25 19:29 pgsql: Merge copies of converting an XID to a FullTransactionId. Noah Misch <noah@leadboat.com>
2025-01-25 19:29 pgsql: Merge copies of converting an XID to a FullTransactionId. Noah Misch <noah@leadboat.com>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox