Content-Length: 270129 | pFad | http://github.com/postgrespro/postgres/commit/75b8d3de989eb88a5960c93cddd88caf5d245024

05 Fix long-obsolete comment. · postgrespro/postgres@75b8d3d · GitHub
Skip to content

Commit 75b8d3d

Browse files
committed
Fix long-obsolete comment.
Commit c94959d fixed DROP OPERATOR to reset oprcom/oprnegate links to the dropped operator; but it missed updating this old comment that claimed we allow such links to dangle.
1 parent 9b4eafc commit 75b8d3d

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/backend/catalog/pg_operator.c

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -827,11 +827,10 @@ makeOperatorDependencies(HeapTuple tuple,
827827

828828
/*
829829
* 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.)
835834
*/
836835

837836
/* Dependency on implementation function */

0 commit comments

Comments
 (0)








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgrespro/postgres/commit/75b8d3de989eb88a5960c93cddd88caf5d245024

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy