Re: enable_resultcache confusion
David Rowley <dgrowleyml@gmail.com>
From: David Rowley <dgrowleyml@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Justin Pryzby <pryzby@telsasoft.com>, Robert Haas <robertmhaas@gmail.com>, "David G. Johnston" <david.g.johnston@gmail.com>,
Bruce Momjian <bruce@momjian.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2021-07-13T00:01:38Z
Lists: pgsql-hackers
On Tue, 13 Jul 2021 at 01:38, Tom Lane <tgl@sss.pgh.pa.us> wrote: > > David Rowley <dgrowleyml@gmail.com> writes: > > On Mon, 12 Jul 2021 at 03:22, Justin Pryzby <pryzby@telsasoft.com> wrote: > >> | This is useful if only a small percentage of rows is checked on > >> | the inner side and is controlled by <xref > >> | linkend="guc-enable-resultcache"/>. > > > You might be right there, but I'm not too sure if I changed that that > > it might cause a mention of the rename to be missed in the changes > > since beta2 notes. > > You need to change it, because IIUC that will be a dangling > cross-reference, causing the v14 docs to fail to build at all. Good point. I'll adjust that for PG14. I plan on pushing the patch to master and PG14 in 24 hours time. If anyone is still on the fence or wishes to object to the name, please let it be known before then. David
Commits
-
Change the name of the Result Cache node to Memoize
- 47ca4836441d 14.0 landed
- 83f4fcc65503 15.0 landed