Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitf3ed536

Browse files
committed
Don't record FDW user mappings as members of extensions.
CreateUserMapping has a recordDependencyOnCurrentExtension call that'sbeen there since extensions were introduced (very possibly my fault).However, there's no support anywhere else for user mappings as membersof extensions, nor are they listed as a possible member object type inthe documentation. Nor does it really seem like a good idea for usermappings to belong to extensions when roles don't. Hence, remove thebogus call.(As we saw in bug #15310, the lack of any pg_dump support for this caseensures that any such membership record would silently disappear duringpg_upgrade. So there's probably no need for us to do anything elseabout cleaning up after this mistake.)Discussion:https://postgr.es/m/27952.1533667213@sss.pgh.pa.us
1 parentf73a313 commitf3ed536

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

‎src/backend/commands/foreigncmds.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1212,8 +1212,12 @@ CreateUserMapping(CreateUserMappingStmt *stmt)
12121212
recordDependencyOnOwner(UserMappingRelationId,umId,useId);
12131213
}
12141214

1215-
/* dependency on extension */
1216-
recordDependencyOnCurrentExtension(&myself, false);
1215+
/*
1216+
* Perhaps someday there should be a recordDependencyOnCurrentExtension
1217+
* call here; but since roles aren't members of extensions, it seems like
1218+
* user mappings shouldn't be either. Note that the grammar and pg_dump
1219+
* would need to be extended too if we change this.
1220+
*/
12171221

12181222
/* Post creation hook for new user mapping */
12191223
InvokeObjectPostCreateHook(UserMappingRelationId,umId,0);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp