Fix long-obsolete comment.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 20 Nov 2022 16:22:22 +0000 (11:22 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 20 Nov 2022 16:22:22 +0000 (11:22 -0500)
Commit c94959d41 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.

src/backend/catalog/pg_operator.c

index 1017f2eed18a91b295cdd42780de45b801757a74..0c0d3f0d0379d5cbdd94a8ad1a52a91eb80c4891 100644 (file)
@@ -827,11 +827,10 @@ makeOperatorDependencies(HeapTuple tuple,
 
        /*
         * NOTE: we do not consider the operator to depend on the associated
-        * operators oprcom and oprnegate. We would not want to delete this
-        * operator if those go away, but only reset the link fields; which is not
-        * a function that the dependency code can presently handle.  (Something
-        * could perhaps be done with objectSubId though.)      For now, it's okay to
-        * let those links dangle if a referenced operator is removed.
+        * operators oprcom and oprnegate.  We do not want to delete this operator
+        * if those go away, but only reset the link fields; which is not a
+        * function that the dependency logic can handle.  (It's taken care of
+        * manually within RemoveOperatorById, instead.)
         */
 
        /* Dependency on implementation function */