pgsql: Skip setting up shared instrumentation for Hash node if notneed - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Skip setting up shared instrumentation for Hash node if notneed
Date
Msg-id E1eiXFK-0008CC-H7@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Skip setting up shared instrumentation for Hash node if not needed.

We don't need to set up the shared space for hash join instrumentation data
if instrumentation hasn't been requested.  Let's follow the example of the
similar Sort node code and save a few cycles by skipping that when we can.

This reverts commit d59ff4ab3 and instead allows us to use the safer choice
of passing noError = false to shm_toc_lookup in ExecHashInitializeWorker,
since if we reach that call there should be a TOC entry to be found.

Thomas Munro

Discussion: https://postgr.es/m/E1ehkoZ-0005uW-43%40gemulon.postgresql.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/05d0f13f0701d84e4e6784da336aabcc2dfc8ade

Modified Files
--------------
src/backend/executor/nodeHash.c | 23 +++++++++++++++++------
1 file changed, 17 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Thomas Munro
Date:
Subject: Re: pgsql: Fix another instance of unsafe coding for shm_toc_lookup failure
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Fix another instance of unsafe coding for shm_toc_lookup failure