Re: Remove unlogged materialized view persistence handling - Mailing list pgsql-hackers

From Yugo NAGATA
Subject Re: Remove unlogged materialized view persistence handling
Date
Msg-id 20241011161804.75bbb4b82180fb2dd5cd2f79@sraoss.co.jp
Whole thread Raw
In response to Re: Remove unlogged materialized view persistence handling  (px shi <spxlyy123@gmail.com>)
Responses Re: Remove unlogged materialized view persistence handling
List pgsql-hackers
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>



pgsql-hackers by date:

Previous
From: Laurenz Albe
Date:
Subject: Re: sunsetting md5 password support
Next
From: Amit Langote
Date:
Subject: Re: generic plans and "initial" pruning