diff options
| author | Peter Eisentraut | 2025-01-24 21:58:13 +0000 |
|---|---|---|
| committer | Peter Eisentraut | 2025-02-01 09:42:58 +0000 |
| commit | 43493cceda2fb75e07a63cec46000407661d505d (patch) | |
| tree | 0c8c0f8d1a109ccd7fb7b4ec45ced857e379f0fe /src/backend/access/brin/brin_validate.c | |
| parent | a5709b5bb293a554913c4b1f6a9c58d1915ba3f7 (diff) | |
Add get_opfamily_name() function
This refactors and simplifies various existing code to make use of the
new function.
Reviewed-by: Mark Dilger <[email protected]>
Discussion: https://www.postgresql.org/message-id/flat/[email protected]
Diffstat (limited to 'src/backend/access/brin/brin_validate.c')
| -rw-r--r-- | src/backend/access/brin/brin_validate.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/src/backend/access/brin/brin_validate.c b/src/backend/access/brin/brin_validate.c index a49a9009626..915b8628b46 100644 --- a/src/backend/access/brin/brin_validate.c +++ b/src/backend/access/brin/brin_validate.c @@ -19,9 +19,9 @@ #include "catalog/pg_amop.h" #include "catalog/pg_amproc.h" #include "catalog/pg_opclass.h" -#include "catalog/pg_opfamily.h" #include "catalog/pg_type.h" #include "utils/builtins.h" +#include "utils/lsyscache.h" #include "utils/regproc.h" #include "utils/syscache.h" @@ -42,8 +42,6 @@ brinvalidate(Oid opclassoid) Oid opfamilyoid; Oid opcintype; char *opclassname; - HeapTuple familytup; - Form_pg_opfamily familyform; char *opfamilyname; CatCList *proclist, *oprlist; @@ -65,12 +63,7 @@ brinvalidate(Oid opclassoid) opclassname = NameStr(classform->opcname); /* Fetch opfamily information */ - familytup = SearchSysCache1(OPFAMILYOID, ObjectIdGetDatum(opfamilyoid)); - if (!HeapTupleIsValid(familytup)) - elog(ERROR, "cache lookup failed for operator family %u", opfamilyoid); - familyform = (Form_pg_opfamily) GETSTRUCT(familytup); - - opfamilyname = NameStr(familyform->opfname); + opfamilyname = get_opfamily_name(opfamilyoid, false); /* Fetch all operators and support functions of the opfamily */ oprlist = SearchSysCacheList1(AMOPSTRATEGY, ObjectIdGetDatum(opfamilyoid)); @@ -274,7 +267,6 @@ brinvalidate(Oid opclassoid) ReleaseCatCacheList(proclist); ReleaseCatCacheList(oprlist); - ReleaseSysCache(familytup); ReleaseSysCache(classtup); return result; |