On Fri, Aug 7, 2020 at 1:12 PM Tom Lane <t...@sss.pgh.pa.us> wrote: > That's a meaningless statement for any one caller. So it needs to be more > like "we expect callers to add and remove temporary before_shmem_exit > callbacks in strict LIFO order".
Sure, that seems fine. > I wonder whether we ought to change the function to complain if the > last list entry doesn't match. We'd have caught this bug sooner > if it did, and it's not very clear why silently doing nothing is > a good idea when there's no match. +1. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company