Re: ALTER TABLE on system catalogs - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: ALTER TABLE on system catalogs
Date
Msg-id 20180820023749.GD7403@paquier.xyz
Whole thread Raw
In response to Re: ALTER TABLE on system catalogs  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Responses Re: ALTER TABLE on system catalogs
Re: ALTER TABLE on system catalogs
List pgsql-hackers
On Fri, Jul 13, 2018 at 11:05:10AM +0200, Peter Eisentraut wrote:
> After reviewing that thread, I think my patch would still be relevant.
> Because the pending proposal is to not add TOAST tables to some catalogs
> such as pg_attribute, so my original use case of allowing ALTER TABLE /
> SET on system catalogs would still be broken for those tables.

Something has happened in this area in the shape of 96cdeae, and the
following catalogs do not have toast tables: pg_class, pg_index,
pg_attribute and pg_largeobject*.  While 0001 proposed upthread is not
really relevant anymore because there is already a test to list which
catalogs do not have a toast table.  For 0002, indeed the patch is still
seems relevant.  The comment block at the beginning of
create_toast_table should be updated.  Some tests would also be nice to
have.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: ATTACH/DETACH PARTITION CONCURRENTLY
Next
From: Tom Lane
Date:
Subject: Re: [FEATURE PATCH] pg_stat_statements with plans (v02)