diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2020-06-09 09:10:14 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2020-06-09 09:39:46 +0200 |
| commit | b1d32d3e3230f00b5baba08f75b4f665c7d6dac6 (patch) | |
| tree | 9b0b6bba1aab417382105d09fc2800f346c7ce2b /src/backend/commands/schemacmds.c | |
| parent | b27c90bbe4a3d607f8fc6703c7183e56e4039acd (diff) | |
Unify drop-by-OID functions
There are a number of Remove${Something}ById() functions that are
essentially identical in structure and only different in which catalog
they are working on. Refactor this to be one generic function. The
information about which oid column, index, etc. to use was already
available in ObjectProperty for most catalogs, in a few cases it was
easily added.
Reviewed-by: Pavel Stehule <pavel.stehule@gmail.com>
Reviewed-by: Robert Haas <robertmhaas@gmail.com>
Discussion: https://www.postgresql.org/message-id/flat/331d9661-1743-857f-1cbb-d5728bcd62cb%402ndquadrant.com
Diffstat (limited to 'src/backend/commands/schemacmds.c')
| -rw-r--r-- | src/backend/commands/schemacmds.c | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/src/backend/commands/schemacmds.c b/src/backend/commands/schemacmds.c index 1e6e731eb58..30138625288 100644 --- a/src/backend/commands/schemacmds.c +++ b/src/backend/commands/schemacmds.c @@ -210,29 +210,6 @@ CreateSchemaCommand(CreateSchemaStmt *stmt, const char *queryString, return namespaceId; } -/* - * Guts of schema deletion. - */ -void -RemoveSchemaById(Oid schemaOid) -{ - Relation relation; - HeapTuple tup; - - relation = table_open(NamespaceRelationId, RowExclusiveLock); - - tup = SearchSysCache1(NAMESPACEOID, - ObjectIdGetDatum(schemaOid)); - if (!HeapTupleIsValid(tup)) /* should not happen */ - elog(ERROR, "cache lookup failed for namespace %u", schemaOid); - - CatalogTupleDelete(relation, &tup->t_self); - - ReleaseSysCache(tup); - - table_close(relation, RowExclusiveLock); -} - /* * Rename schema |