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