John Naylor <johncnaylo...@gmail.com> writes: > I'd prefer just adding "Assert(hentry->event == oldn);" and declaring > hentry PG_USED_FOR_ASSERTS_ONLY.
I'm not aware of any other places where we have Asserts checking that hash_search() honored its contract. Why do we need one here? regards, tom lane