pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter@eisentraut.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Replace get_equal_strategy_number_for_am() by get_equal_strategy
Date: Tue, 10 Dec 2024 11:59:22 +0000
Message-ID: <E1tKyte-0022D7-3P@gemulon.postgresql.org> (raw)

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@illuminatedcomputing.com>
Reviewed-by: vignesh C <vignesh21@gmail.com>
Discussion: https://www.postgresql.org/message-id/flat/CA+renyUApHgSZF9-nd-a0+OPGharLQLO=mDHcY4_qQ0+noCUVg@mail....

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(-)



Message-ID: <E1tKyte-0022D7-3P@gemulon.postgresql.org>
Permalink:  ../E1tKyte-0022D7-3P@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1tKyte-0022D7-3P@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: peter@eisentraut.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Replace get_equal_strategy_number_for_am() by get_equal_strategy
  In-Reply-To: <E1tKyte-0022D7-3P@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox