public inbox for [email protected]
help / color / mirror / Atom feedFrom: Sofia Kopikova <[email protected]>
To: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Subject: Re: Add TOAST support for more system tables
Date: Tue, 18 Jul 2023 14:19:46 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
<[email protected]>
<[email protected]>
On Mon, Jul 17, 2023 at 06:31:04PM -0400, Tom Lane wrote:
> Sofia Kopikova <[email protected]> writes:
>> This patch adds TOAST support for system tables pg_class,
>> pg_attribute and pg_largeobject_metadata, as they include ACL columns,
>> which may be potentially large in size.
> We have been around on this topic before, cf discussion leading up to
> commit 96cdeae07. Allowing toasted data in pg_class or pg_attribute
> seems quite scary to me because of the potential for recursive access,
> particularly during cache-flush scenarios. (That is, you need to be
> able to read those catalogs on the way to fetching a toasted value,
> so how can you be sure that doesn't devolve into an infinite loop?)
Many thanks for your reviews. I'm gonna do research and revise this
feature thoroughly.
I'll set status of the patch to "Waiting on author" for now.
--
Sofia Kopikova
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
view thread (5+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: Add TOAST support for more system tables
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox