From 75b8d3de989eb88a5960c93cddd88caf5d245024 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 20 Nov 2022 11:22:22 -0500 Subject: [PATCH] Fix long-obsolete comment. 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 | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/backend/catalog/pg_operator.c b/src/backend/catalog/pg_operator.c index 1017f2eed1..0c0d3f0d03 100644 --- a/src/backend/catalog/pg_operator.c +++ b/src/backend/catalog/pg_operator.c @@ -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 */