Thread: TransactionIdIsActive() has long been unused
Hi, While looking at Tomas' NUMA patchset I got curious what we use TransactionIdIsActive() for (it's modified in the patchset). Turns out we don't. And haven't for a long while, the last use was removed in commit bb38fb0d43c Author: Heikki Linnakangas <heikki.linnakangas@iki.fi> Date: 2014-05-15 16:37:50 +0300 Fix race condition in preparing a transaction for two-phase commit. Seems like we should just remove TransactionIdIsActive()? Greetings, Andres Freund
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. regards, tom lane
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