Horton

Re: TransactionIdIsActive() has long been unused

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andres Freund <andres@anarazel.de>, pgsql-hackers@postgresql.org, Heikki Linnakangas <hlinnaka@iki.fi>
Date: 2025-07-10T00:52:45Z
Lists: pgsql-hackers 
On Wed, Jul 09, 2025 at 03:46:43PM -0400, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
>> Seems like we should just remove TransactionIdIsActive()?
> 
> +1.  I wondered if any extensions might depend on it, but
> could not find any trace of that in Debian code search.
> And it's a sufficiently odd test (as noted in its comments)
> that one could debate whether using it for an extension's
> purposes would be correct anyway.

+1.
--
Michael

Commits

  1. Remove long-unused TransactionIdIsActive()

  2. Fix race condition in preparing a transaction for two-phase commit.