On Sun, Oct 13, 2019 at 04:18:34PM -0300, Alvaro Herrera wrote:
> True. And we can copy the resulting comment to the other spot.
>
> (FWIW I expect the crash is possible not just in reindex but also in
> CREATE INDEX CONCURRENTLY.)
I need to think about that, but shouldn't we have a way to reproduce
that case rather reliably with an isolation test? The patch looks to
good to me, these are also the two places I spotted yesterday after a
quick lookup. The only other caller is isTempNamespaceInUse() which
does its thing correctly.
--
Michael