Re: setLastTid() and currtid() - Mailing list pgsql-hackers

From Tom Lane
Subject Re: setLastTid() and currtid()
Date
Msg-id 23188.1555005973@sss.pgh.pa.us
Whole thread Raw
In response to Re: setLastTid() and currtid()  (Andres Freund <andres@anarazel.de>)
Responses Re: setLastTid() and currtid()
Re: setLastTid() and currtid()
List pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> On 2019-04-11 13:27:03 -0400, Tom Lane wrote:
>> I think removing it after feature freeze is not something to do,
>> but +1 for nuking it as soon as the v13 branch opens.  Unless
>> there's some important reason we need it to be gone in v12?

> No, I don't think there really is. They're bogus and possibly a bit
> dangerous, but that's not really new.

> I was mostly just reminded of this when Heikki asked me to improve the
> documentation for heap_get_latest_tid/table_get_latest_tid() - and I was
> briefly wondering whether we could just nuke the whole functionality.
> But it's still used in nodeTidscan.c:

Yeah, if we could simplify the tableam API, that would be sufficient
reason to remove the stuff in v12, IMO.  But if it is outside of that
API, I'd counsel waiting till v13.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Retronym: s/magnetic disk/main data/
Next
From: Daniel Gustafsson
Date:
Subject: Re: Enable data checksums by default