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 */