Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
> > Basing an MV on pg_class could always be difficult for pg_upgrade. Maybe
> > that's not a brilliant thing to do in a test (or maybe the test should drop
> > the MV after it's done).
>
> OH. Is that what it's doing? The cause of the failure is immediately
> obvious then. pg_class now lacks a relhaspkey column, but the matview
> is still dependent on one being there.
>
> I concur that this is not well-considered test design.
It seems very strange to me that the test_ddl_deparse database would be
used by a pg_upgrade test, but OK. I'll change it.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services