> == no relispopulated in pg_matviews ==
>
> Client tools like pgAdmin rely on this column to display matview info:

There is (or rather: there wasn't even before), the column is called
pg_matview.ispopulated and it is still there

> column c.relispopulated does not exist
> LINE 6: c.relispopulated AS with_data,
> ^
> HINT: Perhaps you meant to reference the column "c.relpopulated".

What is missing is the relispopulated boolean from pg_class, and I
don't see how that could stay with this approach.

pg_class.relispopulated could only stay if we could update it after
recovery, but before accepting connections (as otherwise it could
become stale for unlogged matviews), and I don't see an easy way to do
that, that's why I implemented this patch in a different way.
Otherwise, it could become stale, that's why the patch removes it.

But looking at pgadmin, it would be easy to patch it in a way that
works with both all earlier versions and this patch.


Reply via email to