From: Peter Eisentraut Date: Sat, 8 Mar 2025 08:37:06 +0000 (+0100) Subject: Make amcanorder independent of amconsistentordering X-Git-Tag: REL_18_BETA1~645 X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=8021c77769e90cc804121d61a1bb7bcc4652d48b;p=postgresql.git Make amcanorder independent of amconsistentordering Follow-up to commit af4002b381d: Make amconsistentordering not depend on amcanorder. Although they are related, they are independent properties. Reported-by: Tom Lane Discussion: https://www.postgresql.org/message-id/flat/E1tngY6-0000UL-2n%40gemulon.postgresql.org --- diff --git a/src/backend/utils/cache/lsyscache.c b/src/backend/utils/cache/lsyscache.c index a712a432938..80c5a3fcfb7 100644 --- a/src/backend/utils/cache/lsyscache.c +++ b/src/backend/utils/cache/lsyscache.c @@ -750,7 +750,7 @@ equality_ops_are_compatible(Oid opno1, Oid opno2) * they belong to the same family.) * * (This is identical to equality_ops_are_compatible(), except that we check - * amcanorder plus amconsistentordering instead of amconsistentequality.) + * amconsistentordering instead of amconsistentequality.) */ bool comparison_ops_are_compatible(Oid opno1, Oid opno2) @@ -782,7 +782,7 @@ comparison_ops_are_compatible(Oid opno1, Oid opno2) { IndexAmRoutine *amroutine = GetIndexAmRoutineByAmId(op_form->amopmethod, false); - if (amroutine->amcanorder && amroutine->amconsistentordering) + if (amroutine->amconsistentordering) { result = true; break;