Re: enable_resultcache confusion - Mailing list pgsql-hackers

From Tom Lane
Subject Re: enable_resultcache confusion
Date
Msg-id 2945408.1626097128@sss.pgh.pa.us
Whole thread Raw
In response to Re: enable_resultcache confusion  (David Rowley <dgrowleyml@gmail.com>)
Responses Re: enable_resultcache confusion  (David Rowley <dgrowleyml@gmail.com>)
List pgsql-hackers
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.

> 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.

I'd suggest replacing it in typedefs.list, since there is unlikely to
be any further update to v14's copy otherwise, and even in HEAD I'm not
sure it'd get updated before we approach the v15 branch.

            regards, tom lane



pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: ATTACH PARTITION locking documentation for DEFAULT partitions
Next
From: Ronan Dunklau
Date:
Subject: Re: [PATCH] Use optimized single-datum tuplesort in ExecSort