Horton

Re: enable_resultcache confusion

David Rowley <dgrowleyml@gmail.com>

View thread View on postgr.es
From: David Rowley <dgrowleyml@gmail.com>
To: Justin Pryzby <pryzby@telsasoft.com>
Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, "David G. Johnston" <david.g.johnston@gmail.com>, Bruce Momjian <bruce@momjian.us>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2021-07-11T23:47:27Z
Lists: pgsql-hackers 
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.

Additionally, I was unsure about touching typedefs.list. In the patch
I changed it, but wasn't too sure if that was the correct thing to do.
In normal circumstances, i.e writing new code, I'd not touch it.

David

Commits

  1. Change the name of the Result Cache node to Memoize