On Thu, Mar 24, 2022 at 05:44:31PM -0400, Tom Lane wrote: > Note that that's basically a workaround for buggy placement of the > OAT hooks, as per previous discussion. I hope that we fix that bug > pretty soon, so it shouldn't really be a factor for the meson conversion.
So, this issue is still listed as an open item. What should we do? From what I get, the caching issues with the namespace lookup hook are not new to v15, they just get exposed by the new test module test_oat_hooks/. FWIW, I would vote against moving around hook calls in back branches as that could cause compatibility problems in existing code relying on them, but it surely is unstable to keep these when recomputing the search_path. A removal from recomputeNamespacePath() implies an addition at the end of fetch_search_path() and fetch_search_path_array(). Perhaps an extra one in RangeVarGetCreationNamespace()? The question is how much of these we want, for example the search hook would be called now even when doing relation-specific checks like RelationIsVisible() and the kind. -- Michael
signature.asc
Description: PGP signature
