Robert Haas <robertmh...@gmail.com> writes: > I still think it would be better to have GetOldestSnapshot() be > smarter and refuse to return the catalog snapshot. For one thing, that > way we'd be testing for the problem case in non-assert builds also.
I was wondering about that too. On the other hand, given that
we know this area is squishy, transforming fails-in-assert-builds
to fails-everywhere is not necessarily desirable.
regards, tom lane
