NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit75b8d3d

committed

Fix long-obsolete comment.

Commitc94959d fixed DROP OPERATOR to reset oprcom/oprnegate linksto the dropped operator; but it missed updating this old comment thatclaimed we allow such links to dangle.

1 parent9b4eafc commit75b8d3dCopy full SHA for 75b8d3d

File tree

1 file changed

-5

lines changed

src/backend/catalog
- pg_operator.c

1 file changed

-5

lines changed

`‎src/backend/catalog/pg_operator.c`

Lines changed: 4 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -827,11 +827,10 @@ makeOperatorDependencies(HeapTuple tuple,`
`827`	`827`
`828`	`828`	`/*`
`829`	`829`	`* NOTE: we do not consider the operator to depend on the associated`
`830`		`- * operators oprcom and oprnegate. We would not want to delete this`
`831`		`- * operator if those go away, but only reset the link fields; which is not`
`832`		`- * a function that the dependency code can presently handle. (Something`
`833`		`- * could perhaps be done with objectSubId though.)For now, it's okay to`
`834`		`- * let those links dangle if a referenced operator is removed.`
	`830`	`+ * operators oprcom and oprnegate. We do not want to delete this operator`
	`831`	`+ * if those go away, but only reset the link fields; which is not a`
	`832`	`+ * function that the dependency logic can handle. (It's taken care of`
	`833`	`+ * manually within RemoveOperatorById, instead.)`
`835`	`834`	`*/`
`836`	`835`
`837`	`836`	`/* Dependency on implementation function */`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit75b8d3d

File tree

1 file changed

1 file changed

`‎src/backend/catalog/pg_operator.c`

0 commit comments