On 2020-11-13 16:57, Tom Lane wrote:
Peter Eisentraut <peter.eisentr...@enterprisedb.com> writes:
On 2020-11-12 23:28, Tom Lane wrote:
I'm on board with pulling these now --- 8.2 to v14 is plenty of
deprecation notice. However, the patch seems incomplete in that
the code support for these is still there -- look for
RTOldContainedByStrategyNumber and RTOldContainsStrategyNumber.
Admittedly, there's not much to be removed except some case labels,
but it still seems like we oughta do that to avoid future confusion.
Yeah, the stuff in gistproc.c should be removed now. But I wonder what
the mentions in brin_inclusion.c are and whether or how they should be
removed.
I think those probably got cargo-culted in there at some point.
They're visibly dead code, because there are no pg_amop entries
for BRIN opclasses with amopstrategy 13 or 14.
I have committed fixes that remove the unused strategy numbers from both
of these code areas.