On Mon, Sep 26, 2022 at 06:05:34PM -0400, Tom Lane wrote: > Yeah. Whether it's efficient or not, seems like it should *work*. > I'm a bit inclined to call this a bug-fix and backpatch it. > > I went looking for other occurrences of this code in places that have > an unregister function, and found one in ResourceOwnerReleaseInternal, > so I think we should fix that too. Also, a comment seems advisable; > that leads me to the attached v2.
LGTM. I have no opinion on back-patching. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
