Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> Commit bc8036fc666a (12 years ago) seems to have introduced an
> unnecessary catalog heap_open/close in order to do syscache accesses.
Huh. Not sure how that got past me, but I agree it's bogus.
> I presume a preliminary version of the patch used sysscans or something.
> I don't think that's necessary, so this patch removes it.
+1, but please collapse up the whitespace too.
regards, tom lane