aboutsummaryrefslogtreecommitdiff
path: root/contrib/btree_gist/btree_oid.c
diff options
context:
space:
mode:
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>2021-04-07 14:33:21 +0300
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>2021-04-07 14:33:21 +0300
commitd92b1cdbab408d8f1299257125c9ae375f3ca644 (patch)
treee75e26cae720ad2f4a0e1d8ef071ae197924c45a /contrib/btree_gist/btree_oid.c
parent9f984ba6d23dc6eecebf479ab1d3f2e550a4e9be (diff)
downloadpostgresql-d92b1cdbab408d8f1299257125c9ae375f3ca644.tar.gz
postgresql-d92b1cdbab408d8f1299257125c9ae375f3ca644.zip
Revert "Add sortsupport for gist_btree opclasses, for faster index builds."
This reverts commit 9f984ba6d23dc6eecebf479ab1d3f2e550a4e9be. It was making the buildfarm unhappy, apparently setting client_min_messages in a regression test produces different output if log_statement='all'. Another issue is that I now suspect the bit sortsupport function was in fact not correct to call byteacmp(). Revert to investigate both of those issues.
Diffstat (limited to 'contrib/btree_gist/btree_oid.c')
-rw-r--r--contrib/btree_gist/btree_oid.c70
1 files changed, 0 insertions, 70 deletions
diff --git a/contrib/btree_gist/btree_oid.c b/contrib/btree_gist/btree_oid.c
index 9b7c546aeeb..3cc7d4245d4 100644
--- a/contrib/btree_gist/btree_oid.c
+++ b/contrib/btree_gist/btree_oid.c
@@ -23,7 +23,6 @@ PG_FUNCTION_INFO_V1(gbt_oid_consistent);
PG_FUNCTION_INFO_V1(gbt_oid_distance);
PG_FUNCTION_INFO_V1(gbt_oid_penalty);
PG_FUNCTION_INFO_V1(gbt_oid_same);
-PG_FUNCTION_INFO_V1(gbt_oid_sortsupport);
static bool
@@ -216,72 +215,3 @@ gbt_oid_same(PG_FUNCTION_ARGS)
*result = gbt_num_same((void *) b1, (void *) b2, &tinfo, fcinfo->flinfo);
PG_RETURN_POINTER(result);
}
-
-static int
-gbt_oid_sort_build_cmp(Datum a, Datum b, SortSupport ssup)
-{
- oidKEY *ia = (oidKEY *) DatumGetPointer(a);
- oidKEY *ib = (oidKEY *) DatumGetPointer(b);
-
- /* for leaf items we expect lower == upper */
- Assert(ia->lower == ia->upper);
- Assert(ib->lower == ib->upper);
-
- if (ia->lower == ib->lower)
- return 0;
-
- return (ia->lower > ib->lower) ? 1 : -1;
-}
-
-static Datum
-gbt_oid_abbrev_convert(Datum original, SortSupport ssup)
-{
- oidKEY *b1 = (oidKEY *) DatumGetPointer(original);
-
- return ObjectIdGetDatum(b1->lower);
-}
-
-static int
-gbt_oid_cmp_abbrev(Datum z1, Datum z2, SortSupport ssup)
-{
- Oid a = DatumGetObjectId(z1);
- Oid b = DatumGetObjectId(z2);
-
- if (a > b)
- return 1;
- else if (a < b)
- return -1;
- else
- return 0;
-}
-
-/*
- * We never consider aborting the abbreviation.
- */
-static bool
-gbt_oid_abbrev_abort(int memtupcount, SortSupport ssup)
-{
- return false;
-}
-
-/*
- * Sort support routine for fast GiST index build by sorting.
- */
-Datum
-gbt_oid_sortsupport(PG_FUNCTION_ARGS)
-{
- SortSupport ssup = (SortSupport) PG_GETARG_POINTER(0);
-
- if (ssup->abbreviate)
- {
- ssup->comparator = gbt_oid_cmp_abbrev;
- ssup->abbrev_converter = gbt_oid_abbrev_convert;
- ssup->abbrev_abort = gbt_oid_abbrev_abort;
- ssup->abbrev_full_comparator = gbt_oid_sort_build_cmp;
- }
- else
- {
- ssup->comparator = gbt_oid_sort_build_cmp;
- }
- PG_RETURN_VOID();
-}