Re: TransactionIdIsActive() has long been unused

2025-07-12 Thread Andres Freund
Hi, On 2025-07-10 09:52:45 +0900, Michael Paquier wrote: > On Wed, Jul 09, 2025 at 03:46:43PM -0400, Tom Lane wrote: > > Andres Freund writes: > >> Seems like we should just remove TransactionIdIsActive()? > > > > +1. I wondered if any extensions might depend on it, but > > could not find any t

Re: TransactionIdIsActive() has long been unused

2025-07-09 Thread Michael Paquier
On Wed, Jul 09, 2025 at 03:46:43PM -0400, Tom Lane wrote: > Andres Freund 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

Re: TransactionIdIsActive() has long been unused

2025-07-09 Thread Tom Lane
Andres Freund 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 fo