Re: missing toast table for pg_policy

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andres Freund <andres(at)anarazel(dot)de>, John Naylor <jcnaylor(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: missing toast table for pg_policy
Date: 2018-07-19 23:46:50
Message-ID: 20180719234650.GB7023@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jul 19, 2018 at 07:18:32PM -0400, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
>> FWIW, I was off the last few days. I personally think the reasoning to
>> leave out pg_class, pg_index etc. is bad. We should just make them work
>> and create toast tables as well.
>
> If it's easy to make those work and keep them working, then sure, but
> I have my doubts. I remain afraid of circular accesses occurring only
> in strange corner cases ...

I have found the argument about circular dependencies rather sensible
FWIW. So at the end it seems to me that we would not want to add toast
tables for those catalogs.

>> It's definitely not right that "those
>> relations have no reason to use a toast table anyway." as the commit
>> message states, given relacl, reloptions and relpartbound.
>
> I wonder whether we shouldn't have handled ACLs through something more
> like the pg_description solution, ie keep them all in one catalog with
> a (classoid, objoid) primary key.

That could be nice, but separate from the fact of adding a toast table
to it?
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2018-07-19 23:50:06 Re: missing toast table for pg_policy
Previous Message Tom Lane 2018-07-19 23:18:32 Re: missing toast table for pg_policy