From: Tom Lane <tgl@sss.pgh.pa.us>
To: Heikki Linnakangas <hlinnaka@iki.fi>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Allow table AM's to cache stuff in relcache
Date: Fri, 14 Jun 2019 11:40:36 -0400
Message-ID: <28934.1560526836@sss.pgh.pa.us> (raw)
In-Reply-To: <dc0a5fe7-9adc-84f7-40e8-a3cde9529a5f@iki.fi>
References: <dc0a5fe7-9adc-84f7-40e8-a3cde9529a5f@iki.fi>
Heikki Linnakangas <hlinnaka@iki.fi> writes:
> Index AM's can cache stuff in RelationData->rd_amcache. In the zedstore
> table AM we've been hacking on, I'd like to also use rd_amcache to cache
> some information, but that's not currently possible, because rd_amcache
> can only be used by index AMs, not table AMs.
> Attached patch allows rd_amcache to also be used by table AMs.
Seems reasonable.
> In the patch, I documented that rd_amcache must be allocated in
> CacheMemoryContext, or in rd_indexcxt if it's an index. It works, but
> it's a bit weird.
Given the way the patch is implemented, it doesn't really matter which
context it's in, does it? The retail pfree is inessential but also
harmless, if rd_amcache is in rd_indexcxt. So we could take out the
"must". I think it's slightly preferable to use rd_indexcxt if available,
to reduce the amount of loose junk in CacheMemoryContext.
> It would nice to have one memory context in every
> relcache entry, to hold all the stuff related to it, including
> rd_amcache. In other words, it would be nice if we had "rd_indexcxt" for
> tables, too, not just indexes. That would allow tracking memory usage
> more accurately, if you're debugging an out of memory situation for example.
We had some discussion related to that in the "hyrax
vs. RelationBuildPartitionDesc" thread. I'm not quite sure where
we'll settle on that, but some redesign seems inevitable.
regards, tom lane
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: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, hlinnaka@iki.fi
Subject: Re: Allow table AM's to cache stuff in relcache
In-Reply-To: <28934.1560526836@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox