pgsql: Replace get_equal_strategy_number_for_am() by get_equal_strategy

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Replace get_equal_strategy_number_for_am() by get_equal_strategy
Date: 2024-12-10 11:59:22
Message-ID: E1tKyte-0022D7-3P@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Replace get_equal_strategy_number_for_am() by get_equal_strategy_number()

get_equal_strategy_number_for_am() gets the equal strategy number for
an AM. This currently only supports btree and hash. In the more
general case, this also depends on the operator class (see for example
GistTranslateStratnum()). To support that, replace this function with
get_equal_strategy_number() that takes an opclass and derives it from
there. (This function already existed before as a static function, so
the signature is kept for simplicity.)

This patch is only a refactoring, it doesn't add support for other
index AMs such as gist. This will be done separately.

Reviewed-by: Paul Jungwirth <pj(at)illuminatedcomputing(dot)com>
Reviewed-by: vignesh C <vignesh21(at)gmail(dot)com>
Discussion: https://www.postgresql.org/message-id/flat/CA+renyUApHgSZF9-nd-a0+OPGharLQLO=mDHcY4_qQ0+noCUVg(at)mail(dot)gmail(dot)com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a2a475b011cf7c25f5a09574def35b335af844ac

Modified Files
--------------
src/backend/executor/execReplication.c | 17 +++--------------
src/backend/replication/logical/relation.c | 19 ++++++++++++++-----
src/include/executor/executor.h | 2 +-
3 files changed, 18 insertions(+), 20 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2024-12-10 12:17:20 pgsql: Make the conditions in IsIndexUsableForReplicaIdentityFull() mor
Previous Message Peter Eisentraut 2024-12-10 11:34:52 pgsql: Improve internal logical replication error for missing equality