diff options
author | Peter Eisentraut | 2025-02-27 16:03:31 +0000 |
---|---|---|
committer | Peter Eisentraut | 2025-02-27 16:03:31 +0000 |
commit | ce62f2f2a0a48d021f250ba84dfcab5d45ddc914 (patch) | |
tree | dd73a4a521f961961ce8c991363b3425e9e4741d /src/backend/utils/cache | |
parent | 6eb8a1a4f90c542d7ce9dcc381528fcb81390ab9 (diff) |
Generalize hash and ordering support in amapi
Stop comparing access method OID values against HASH_AM_OID and
BTREE_AM_OID, and instead check the IndexAmRoutine for an index to see
if it advertises its ability to perform the necessary ordering,
hashing, or cross-type comparing functionality. A field amcanorder
already existed, this uses it more widely. Fields amcanhash and
amcancrosscompare are added for the other purposes.
Author: Mark Dilger <mark.dilger@enterprisedb.com>
Discussion: https://www.postgresql.org/message-id/flat/E72EAA49-354D-4C2E-8EB9-255197F55330@enterprisedb.com
Diffstat (limited to 'src/backend/utils/cache')
-rw-r--r-- | src/backend/utils/cache/lsyscache.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/utils/cache/lsyscache.c b/src/backend/utils/cache/lsyscache.c index bcfa5cb4add..7bd476f3de7 100644 --- a/src/backend/utils/cache/lsyscache.c +++ b/src/backend/utils/cache/lsyscache.c @@ -716,10 +716,9 @@ equality_ops_are_compatible(Oid opno1, Oid opno2) { HeapTuple op_tuple = &catlist->members[i]->tuple; Form_pg_amop op_form = (Form_pg_amop) GETSTRUCT(op_tuple); + IndexAmRoutine *amroutine = GetIndexAmRoutineByAmId(op_form->amopmethod, false); - /* must be btree or hash */ - if (op_form->amopmethod == BTREE_AM_OID || - op_form->amopmethod == HASH_AM_OID) + if (amroutine->amcancrosscompare) { if (op_in_opfamily(opno2, op_form->amopfamily)) { @@ -767,8 +766,9 @@ comparison_ops_are_compatible(Oid opno1, Oid opno2) { HeapTuple op_tuple = &catlist->members[i]->tuple; Form_pg_amop op_form = (Form_pg_amop) GETSTRUCT(op_tuple); + IndexAmRoutine *amroutine = GetIndexAmRoutineByAmId(op_form->amopmethod, false); - if (op_form->amopmethod == BTREE_AM_OID) + if (amroutine->amcanorder && amroutine->amcancrosscompare) { if (op_in_opfamily(opno2, op_form->amopfamily)) { |