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

From Fujii Masao
Subject Re: Remove unlogged materialized view persistence handling
Date
Msg-id ec599969-fbc1-4a0b-9d4a-9a77acab1fb8@oss.nttdata.com
Whole thread Raw
In response to Re: Remove unlogged materialized view persistence handling  (Yugo NAGATA <nagata@sraoss.co.jp>)
Responses Re: Remove unlogged materialized view persistence handling
List pgsql-hackers

On 2024/10/11 16:18, Yugo NAGATA wrote:
> 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.

Yes. I initially thought this code was kept intentionally because
some older PostgreSQL versions supported unlogged materialized views,
and even the latest psql might need to display them. However,
this isn't the case. Unlogged materialized views were added and
removed before the v9.3 release and were never supported in
any official release. So there seems no need to keep this code.

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION




pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: pg_upgrade check for invalid databases
Next
From: Yugo NAGATA
Date:
Subject: Re: CREATE INDEX regression in 17 RC1 or expected behavior?