Andrey Lepikhov <a.lepik...@postgrespro.ru> writes:
> But core jumbling code is good, fast and much easier in support.
It won't be fast once you stick a bunch of catalog lookups into it.
I think this is fine as an extension, but it has no chance of being
accepted in core, just on performance grounds.
(I'm also not sure that the query ID calculation code is always/only
invoked in contexts where it's safe to do catalog accesses.)
A bigger issue is that query ID stability isn't something we are going
to promise on a large scale --- for example, what if a new release adds
some new fields to struct Query? So I'm not sure that "query IDs should
survive dump/reload" is a useful goal to consider. It's certainly not
something that could be reached by anything even remotely like the
existing code.
regards, tom lane
