Horton

Re: [PATCH] remove deprecated v8.2 containment operators

Peter Eisentraut <peter.eisentraut@enterprisedb.com>

View thread View on postgr.es
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Justin Pryzby <pryzby@telsasoft.com>, Heikki Linnakangas <hlinnaka@iki.fi>, Stephen Frost <sfrost@snowman.net>, Ian Lawrence Barwick <barwick@gmail.com>, pgsql-hackers@postgresql.org
Date: 2020-11-13T07:26:28Z
Lists: pgsql-hackers 
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.

Commits

  1. Doc: remove obsolete entries in table of BRIN strategy numbers.

  2. Remove deprecated containment operators for contrib types.

  3. Remove unused and deprecated strategy numbers from BRIN code

  4. Remove code handling removed deprecated containment operators

  5. Remove deprecated containment operators for built-in types

  6. Remove use of deprecated containment operators in tests