On Fri, 11 Oct 2024 14:44:43 +0800
px shi <spxlyy123@gmail.com> wrote:
> >
> > +1
> >
> > Materialized view is introduced by 3bf3ab8c563 and at that UNLOGGED was
> > allowed, and it is disallowed by another commit 3223b25ff73. However,
> > it seems that the tab-complement is missed to fixed.
Sorry, the comment above was not correct.
The fix was about pasl's \d meta-command, not tab-complement.
- if (tableinfo.relpersistence == 'u')
- printfPQExpBuffer(&title, _("Unlogged materialized view \"%s.%s\""),
- schemaname, relationname);
- else
- printfPQExpBuffer(&title, _("Materialized view \"%s.%s\""),
- schemaname, relationname);
+ printfPQExpBuffer(&title, _("Materialized view \"%s.%s\""),
+
"Unlogged materialized view" is never printed since it cannot be created from the beginning
of materialized view support, so this is a dead code to be removed.
> >
> > The Assert for RELKIND_MATVIEW in heapam_hander.c was introduced in
> > d25f519107b
> > after that, but I think it can be also removed.
> >
>
> Thank you for your feedback and support!
>
> I also add this as a post in Commit Fest [0]
>
>
> [0] https://commitfest.postgresql.org/50/5292/
I marked it as Ready-for-Committer.
Regards,
Yugo Nagata
--
Yugo NAGATA <nagata@sraoss.co.jp>
