Remove <@ from contrib/intarray's GiST operator classes.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 8 Aug 2020 21:26:29 +0000 (17:26 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 8 Aug 2020 21:26:29 +0000 (17:26 -0400)
commit20e7e1fe316467720d8d062e1a1429f798fc31bf
tree631137904f5bf5d3ebfc07d7fe0ccbcb7c67269e
parent39132b784aeaaacf5ddfb5c35b6e29a6926f4345
Remove <@ from contrib/intarray's GiST operator classes.

Since commit efc77cf5f, an indexed query using <@ has required a
full-index scan, so that it actually performs worse than a plain seqscan
would do.  As I noted at the time, we'd be better off to not treat <@ as
being indexable by such indexes at all; and that's what this patch does.

It would have been difficult to remove these opclass members without
dropping the whole opclass before commit 9f9682783 fixed GiST opclass
member dependency rules, but now it's quite simple, so let's do it.

I left the existing support code in place for the time being, with
comments noting it's now unreachable.  At some point, perhaps we should
remove that code in favor of throwing an error telling people to upgrade
the extension version.

Discussion: https://postgr.es/m/2176979.1596389859@sss.pgh.pa.us
Discussion: https://postgr.es/m/458.1565114141@sss.pgh.pa.us
contrib/intarray/Makefile
contrib/intarray/_int_gist.c
contrib/intarray/_intbig_gist.c
contrib/intarray/intarray--1.3--1.4.sql [new file with mode: 0644]
contrib/intarray/intarray.control
doc/src/sgml/intarray.sgml