database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-27 22:56:53 +03:00
Code

Rename GistTranslateStratnum() to GistTranslateCompareType()

Browse Source 
Follow up to commit 630f9a43cec.  The previous name had become
confusing, because it doesn't actually translate a strategy number but
a CompareType into a strategy number.  We might add the inverse at
some point, which would then probably be called something like
GistTranslateStratnum.

Reviewed-by: Mark Dilger <mark.dilger@enterprisedb.com>
Discussion: https://www.postgresql.org/message-id/flat/E72EAA49-354D-4C2E-8EB9-255197F55330@enterprisedb.com
This commit is contained in:
Peter Eisentraut 2025-02-01 10:18:46 +01:00
parent 2452e71ff2
commit a5709b5bb2
5 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/backend/access/gist/gistutil.c
View File

@ -1095,7 +1095,7 @@ gist_stratnum_common(PG_FUNCTION_ARGS)
* Returns InvalidStrategy if the function is not defined. * Returns InvalidStrategy if the function is not defined.
*/ */
StrategyNumber StrategyNumber
GistTranslateStratnum(Oid opclass, CompareType cmptype) GistTranslateCompareType(Oid opclass, CompareType cmptype)
{ {
Oid opfamily; Oid opfamily;
Oid opcintype; Oid opcintype;

2
src/backend/commands/indexcmds.c
View File

@ -2455,7 +2455,7 @@ GetOperatorFromCompareType(Oid opclass, Oid rhstype, CompareType cmptype,
* For now we only need GiST support, but this could support other * For now we only need GiST support, but this could support other
* indexams if we wanted. * indexams if we wanted.
*/ */
*strat = GistTranslateStratnum(opclass, cmptype); *strat = GistTranslateCompareType(opclass, cmptype);
if (*strat == InvalidStrategy) if (*strat == InvalidStrategy)
{ {
HeapTuple tuple; HeapTuple tuple;

4
src/backend/commands/tablecmds.c
View File

@ -10016,7 +10016,7 @@ ATAddForeignKeyConstraint(List **wqueue, AlteredTableInfo *tab, Relation rel,
* ask the opclass what number it actually uses instead of our RT* * ask the opclass what number it actually uses instead of our RT*
* constants. * constants.
*/ */
eqstrategy = GistTranslateStratnum(opclasses[i], cmptype); eqstrategy = GistTranslateCompareType(opclasses[i], cmptype);
if (eqstrategy == InvalidStrategy) if (eqstrategy == InvalidStrategy)
{ {
HeapTuple tuple; HeapTuple tuple;
@ -10041,7 +10041,7 @@ ATAddForeignKeyConstraint(List **wqueue, AlteredTableInfo *tab, Relation rel,
* other index AMs support unique indexes. If we ever did have * other index AMs support unique indexes. If we ever did have
* other types of unique indexes, we'd need a way to determine * other types of unique indexes, we'd need a way to determine
* which operator strategy number is equality. (We could use * which operator strategy number is equality. (We could use
* something like GistTranslateStratnum.) * something like GistTranslateCompareType.)
*/ */
if (amid != BTREE_AM_OID) if (amid != BTREE_AM_OID)
elog(ERROR, "only b-tree indexes are supported for foreign keys"); elog(ERROR, "only b-tree indexes are supported for foreign keys");

2
src/backend/executor/execReplication.c
View File

@ -57,7 +57,7 @@ get_equal_strategy_number(Oid opclass)
ret = HTEqualStrategyNumber; ret = HTEqualStrategyNumber;
break; break;
case GIST_AM_OID: case GIST_AM_OID:
ret = GistTranslateStratnum(opclass, COMPARE_EQ); ret = GistTranslateCompareType(opclass, COMPARE_EQ);
break; break;
default: default:
ret = InvalidStrategy; ret = InvalidStrategy;

2
src/include/access/gist.h
View File

@ -248,6 +248,6 @@ typedef struct
do { (e).key = (k); (e).rel = (r); (e).page = (pg); \ do { (e).key = (k); (e).rel = (r); (e).page = (pg); \
(e).offset = (o); (e).leafkey = (l); } while (0) (e).offset = (o); (e).leafkey = (l); } while (0)
extern StrategyNumber GistTranslateStratnum(Oid opclass, CompareType cmp); extern StrategyNumber GistTranslateCompareType(Oid opclass, CompareType cmptype);
#endif /* GIST_H */ #endif /* GIST_H */