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

Commit5fc7039

Browse files
committed
Add ALTER .. NO DEPENDS ON
Commitf2fcad2 (9.6 era) added the ability to mark objects asdependent an extension, but forgot to add a way for such dependencies tobe removed. This commit fixes that oversight.Strictly speaking this should be backpatched to 9.6, but due to lack ofdemand we're not doing so at this time.Discussion:https://postgr.es/m/20200217225333.GA30974@alvherre.pgsqlReviewed-by: ahsan hadi <ahsan.hadi@gmail.com>Reviewed-by: Ibrar Ahmed <ibrar.ahmad@gmail.com>Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
1 parent4157f73 commit5fc7039

File tree

13 files changed

+168
-36
lines changed

13 files changed

+168
-36
lines changed

‎doc/src/sgml/ref/alter_function.sgml‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ ALTER FUNCTION <replaceable>name</replaceable> [ ( [ [ <replaceable class="param
3030
ALTER FUNCTION <replaceable>name</replaceable> [ ( [ [ <replaceable class="parameter">argmode</replaceable> ] [ <replaceable class="parameter">argname</replaceable> ] <replaceable class="parameter">argtype</replaceable> [, ...] ] ) ]
3131
SET SCHEMA <replaceable>new_schema</replaceable>
3232
ALTER FUNCTION <replaceable>name</replaceable> [ ( [ [ <replaceable class="parameter">argmode</replaceable> ] [ <replaceable class="parameter">argname</replaceable> ] <replaceable class="parameter">argtype</replaceable> [, ...] ] ) ]
33-
DEPENDS ON EXTENSION <replaceable>extension_name</replaceable>
33+
[ NO ]DEPENDS ON EXTENSION <replaceable>extension_name</replaceable>
3434

3535
<phrase>where <replaceable class="parameter">action</replaceable> is one of:</phrase>
3636

@@ -153,10 +153,14 @@ ALTER FUNCTION <replaceable>name</replaceable> [ ( [ [ <replaceable class="param
153153
</varlistentry>
154154

155155
<varlistentry>
156-
<term><replaceable class="parameter">extension_name</replaceable></term>
156+
<term><literal>DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable></literal></term>
157+
<term><literal>NO DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable></literal></term>
157158
<listitem>
158159
<para>
159-
The name of the extension that the function is to depend on.
160+
This form marks the function as dependent on the extension, or no longer
161+
dependent on that extension if <literal>NO</literal> is specified.
162+
A function that's marked as dependent on an extension is automatically
163+
dropped when the extension is dropped.
160164
</para>
161165
</listitem>
162166
</varlistentry>

‎doc/src/sgml/ref/alter_index.sgml‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,11 +100,14 @@ ALTER INDEX ALL IN TABLESPACE <replaceable class="parameter">name</replaceable>
100100
</varlistentry>
101101

102102
<varlistentry>
103-
<term><literal>DEPENDS ON EXTENSION</literal></term>
103+
<term><literal>DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable></literal></term>
104+
<term><literal>NO DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable></literal></term>
104105
<listitem>
105106
<para>
106-
This form marks the index as dependent on the extension, such that if the
107-
extension is dropped, the index will automatically be dropped as well.
107+
This form marks the index as dependent on the extension, or no longer
108+
dependent on that extension if <literal>NO</literal> is specified.
109+
An index that's marked as dependent on an extension is automatically
110+
dropped when the extension is dropped.
108111
</para>
109112
</listitem>
110113
</varlistentry>

‎doc/src/sgml/ref/alter_materialized_view.sgml‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -68,12 +68,6 @@ ALTER MATERIALIZED VIEW ALL IN TABLESPACE <replaceable class="parameter">name</r
6868
anyway.)
6969
</para>
7070

71-
<para>
72-
The <literal>DEPENDS ON EXTENSION</literal> form marks the materialized view
73-
as dependent on an extension, such that the materialized view will
74-
automatically be dropped if the extension is dropped.
75-
</para>
76-
7771
<para>
7872
The statement subforms and actions available for
7973
<command>ALTER MATERIALIZED VIEW</command> are a subset of those available
@@ -110,7 +104,10 @@ ALTER MATERIALIZED VIEW ALL IN TABLESPACE <replaceable class="parameter">name</r
110104
<term><replaceable class="parameter">extension_name</replaceable></term>
111105
<listitem>
112106
<para>
113-
The name of the extension that the materialized view is to depend on.
107+
The name of the extension that the materialized view is to depend on (or no longer
108+
dependent on, if <literal>NO</literal> is specified). A materialized view
109+
that's marked as dependent on an extension is automatically dropped when
110+
the extension is dropped.
114111
</para>
115112
</listitem>
116113
</varlistentry>

‎doc/src/sgml/ref/alter_trigger.sgml‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ PostgreSQL documentation
2222
<refsynopsisdiv>
2323
<synopsis>
2424
ALTER TRIGGER <replaceable class="parameter">name</replaceable> ON <replaceable class="parameter">table_name</replaceable> RENAME TO <replaceable class="parameter">new_name</replaceable>
25-
ALTER TRIGGER <replaceable class="parameter">name</replaceable> ON <replaceable class="parameter">table_name</replaceable> DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable>
25+
ALTER TRIGGER <replaceable class="parameter">name</replaceable> ON <replaceable class="parameter">table_name</replaceable>[ NO ]DEPENDS ON EXTENSION <replaceable class="parameter">extension_name</replaceable>
2626
</synopsis>
2727
</refsynopsisdiv>
2828

@@ -78,7 +78,10 @@ ALTER TRIGGER <replaceable class="parameter">name</replaceable> ON <replaceable
7878
<term><replaceable class="parameter">extension_name</replaceable></term>
7979
<listitem>
8080
<para>
81-
The name of the extension that the trigger is to depend on.
81+
The name of the extension that the trigger is to depend on (or no longer
82+
dependent on, if <literal>NO</literal> is specified). A trigger
83+
that's marked as dependent on an extension is automatically dropped when
84+
the extension is dropped.
8285
</para>
8386
</listitem>
8487
</varlistentry>

‎src/backend/catalog/pg_depend.c‎

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,55 @@ deleteDependencyRecordsForClass(Oid classId, Oid objectId,
278278
returncount;
279279
}
280280

281+
/*
282+
* deleteDependencyRecordsForSpecific -- delete all records with given depender
283+
* classId/objectId, dependee classId/objectId, of the given deptype.
284+
* Returns the number of records deleted.
285+
*/
286+
long
287+
deleteDependencyRecordsForSpecific(OidclassId,OidobjectId,chardeptype,
288+
OidrefclassId,OidrefobjectId)
289+
{
290+
longcount=0;
291+
RelationdepRel;
292+
ScanKeyDatakey[2];
293+
SysScanDescscan;
294+
HeapTupletup;
295+
296+
depRel=table_open(DependRelationId,RowExclusiveLock);
297+
298+
ScanKeyInit(&key[0],
299+
Anum_pg_depend_classid,
300+
BTEqualStrategyNumber,F_OIDEQ,
301+
ObjectIdGetDatum(classId));
302+
ScanKeyInit(&key[1],
303+
Anum_pg_depend_objid,
304+
BTEqualStrategyNumber,F_OIDEQ,
305+
ObjectIdGetDatum(objectId));
306+
307+
scan=systable_beginscan(depRel,DependDependerIndexId, true,
308+
NULL,2,key);
309+
310+
while (HeapTupleIsValid(tup=systable_getnext(scan)))
311+
{
312+
Form_pg_dependdepform= (Form_pg_depend)GETSTRUCT(tup);
313+
314+
if (depform->refclassid==refclassId&&
315+
depform->refobjid==refobjectId&&
316+
depform->deptype==deptype)
317+
{
318+
CatalogTupleDelete(depRel,&tup->t_self);
319+
count++;
320+
}
321+
}
322+
323+
systable_endscan(scan);
324+
325+
table_close(depRel,RowExclusiveLock);
326+
327+
returncount;
328+
}
329+
281330
/*
282331
* Adjust dependency record(s) to point to a different object of the same type
283332
*

‎src/backend/commands/alter.c‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -421,7 +421,7 @@ ExecRenameStmt(RenameStmt *stmt)
421421
}
422422

423423
/*
424-
* Executes an ALTER OBJECT / DEPENDS ON[EXTENSION] statement.
424+
* Executes an ALTER OBJECT /[NO]DEPENDS ON EXTENSION statement.
425425
*
426426
* Return value is the address of the altered object. refAddress is an output
427427
* argument which, if not null, receives the address of the object that the
@@ -433,7 +433,6 @@ ExecAlterObjectDependsStmt(AlterObjectDependsStmt *stmt, ObjectAddress *refAddre
433433
ObjectAddressaddress;
434434
ObjectAddressrefAddr;
435435
Relationrel;
436-
List*currexts;
437436

438437
address=
439438
get_object_address_rv(stmt->objectType,stmt->relation, (List*)stmt->object,
@@ -463,11 +462,22 @@ ExecAlterObjectDependsStmt(AlterObjectDependsStmt *stmt, ObjectAddress *refAddre
463462
if (refAddress)
464463
*refAddress=refAddr;
465464

466-
/* Avoid duplicates */
467-
currexts=getAutoExtensionsOfObject(address.classId,
468-
address.objectId);
469-
if (!list_member_oid(currexts,refAddr.objectId))
470-
recordDependencyOn(&address,&refAddr,DEPENDENCY_AUTO_EXTENSION);
465+
if (stmt->remove)
466+
{
467+
deleteDependencyRecordsForSpecific(address.classId,address.objectId,
468+
DEPENDENCY_AUTO_EXTENSION,
469+
refAddr.classId,refAddr.objectId);
470+
}
471+
else
472+
{
473+
List*currexts;
474+
475+
/* Avoid duplicates */
476+
currexts=getAutoExtensionsOfObject(address.classId,
477+
address.objectId);
478+
if (!list_member_oid(currexts,refAddr.objectId))
479+
recordDependencyOn(&address,&refAddr,DEPENDENCY_AUTO_EXTENSION);
480+
}
471481

472482
returnaddress;
473483
}

‎src/backend/nodes/copyfuncs.c‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3638,6 +3638,7 @@ _copyAlterObjectDependsStmt(const AlterObjectDependsStmt *from)
36383638
COPY_NODE_FIELD(relation);
36393639
COPY_NODE_FIELD(object);
36403640
COPY_NODE_FIELD(extname);
3641+
COPY_SCALAR_FIELD(remove);
36413642

36423643
returnnewnode;
36433644
}

‎src/backend/nodes/equalfuncs.c‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1449,6 +1449,7 @@ _equalAlterObjectDependsStmt(const AlterObjectDependsStmt *a, const AlterObjectD
14491449
COMPARE_NODE_FIELD(relation);
14501450
COMPARE_NODE_FIELD(object);
14511451
COMPARE_NODE_FIELD(extname);
1452+
COMPARE_SCALAR_FIELD(remove);
14521453

14531454
return true;
14541455
}

‎src/backend/parser/gram.y‎

Lines changed: 23 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ static Node *makeRecursiveViewSelect(char *relname, List *aliases, Node *query);
320320
%type<list>vac_analyze_option_list
321321
%type<node>vac_analyze_option_arg
322322
%type<defelt>drop_option
323-
%type<boolean>opt_or_replace
323+
%type<boolean>opt_or_replaceopt_no
324324
opt_grant_grant_optionopt_grant_admin_option
325325
opt_nowaitopt_if_existsopt_with_data
326326
opt_transaction_chain
@@ -9053,57 +9053,67 @@ opt_set_data: SET DATA_P{ $$ = 1; }
90539053
*****************************************************************************/
90549054

90559055
AlterObjectDependsStmt:
9056-
ALTERFUNCTIONfunction_with_argtypesDEPENDSONEXTENSIONname
9056+
ALTERFUNCTIONfunction_with_argtypesopt_noDEPENDSONEXTENSIONname
90579057
{
90589058
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
90599059
n->objectType = OBJECT_FUNCTION;
90609060
n->object = (Node *)$3;
9061-
n->extname = makeString($7);
9061+
n->extname = makeString($8);
9062+
n->remove =$4;
90629063
$$ = (Node *)n;
90639064
}
9064-
|ALTERPROCEDUREfunction_with_argtypesDEPENDSONEXTENSIONname
9065+
|ALTERPROCEDUREfunction_with_argtypesopt_noDEPENDSONEXTENSIONname
90659066
{
90669067
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
90679068
n->objectType = OBJECT_PROCEDURE;
90689069
n->object = (Node *)$3;
9069-
n->extname = makeString($7);
9070+
n->extname = makeString($8);
9071+
n->remove =$4;
90709072
$$ = (Node *)n;
90719073
}
9072-
|ALTERROUTINEfunction_with_argtypesDEPENDSONEXTENSIONname
9074+
|ALTERROUTINEfunction_with_argtypesopt_noDEPENDSONEXTENSIONname
90739075
{
90749076
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
90759077
n->objectType = OBJECT_ROUTINE;
90769078
n->object = (Node *)$3;
9077-
n->extname = makeString($7);
9079+
n->extname = makeString($8);
9080+
n->remove =$4;
90789081
$$ = (Node *)n;
90799082
}
9080-
|ALTERTRIGGERnameONqualified_nameDEPENDSONEXTENSIONname
9083+
|ALTERTRIGGERnameONqualified_nameopt_noDEPENDSONEXTENSIONname
90819084
{
90829085
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
90839086
n->objectType = OBJECT_TRIGGER;
90849087
n->relation =$5;
90859088
n->object = (Node *) list_make1(makeString($3));
9086-
n->extname = makeString($9);
9089+
n->extname = makeString($10);
9090+
n->remove =$6;
90879091
$$ = (Node *)n;
90889092
}
9089-
|ALTERMATERIALIZEDVIEWqualified_nameDEPENDSONEXTENSIONname
9093+
|ALTERMATERIALIZEDVIEWqualified_nameopt_noDEPENDSONEXTENSIONname
90909094
{
90919095
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
90929096
n->objectType = OBJECT_MATVIEW;
90939097
n->relation =$4;
9094-
n->extname = makeString($8);
9098+
n->extname = makeString($9);
9099+
n->remove =$5;
90959100
$$ = (Node *)n;
90969101
}
9097-
|ALTERINDEXqualified_nameDEPENDSONEXTENSIONname
9102+
|ALTERINDEXqualified_nameopt_noDEPENDSONEXTENSIONname
90989103
{
90999104
AlterObjectDependsStmt *n = makeNode(AlterObjectDependsStmt);
91009105
n->objectType = OBJECT_INDEX;
91019106
n->relation =$3;
9102-
n->extname = makeString($7);
9107+
n->extname = makeString($8);
9108+
n->remove =$4;
91039109
$$ = (Node *)n;
91049110
}
91059111
;
91069112

9113+
opt_no:NO{$$ =true; }
9114+
|/* EMPTY*/{$$ =false;}
9115+
;
9116+
91079117
/*****************************************************************************
91089118
*
91099119
* ALTER THING name SET SCHEMA name

‎src/include/catalog/dependency.h‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,10 @@ extern long deleteDependencyRecordsFor(Oid classId, Oid objectId,
196196
externlongdeleteDependencyRecordsForClass(OidclassId,OidobjectId,
197197
OidrefclassId,chardeptype);
198198

199+
externlongdeleteDependencyRecordsForSpecific(OidclassId,OidobjectId,
200+
chardeptype,
201+
OidrefclassId,OidrefobjectId);
202+
199203
externlongchangeDependencyFor(OidclassId,OidobjectId,
200204
OidrefClassId,OidoldRefObjectId,
201205
OidnewRefObjectId);

0 commit comments

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