From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Remove stray references to lefttree/righttree in the executor. |
Date: | 2022-07-07 15:24:04 |
Message-ID: | E1o9TMI-001ent-TD@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Remove stray references to lefttree/righttree in the executor.
The general convention in the executor is to refer to child plans
and planstates via the outerPlan[State] and innerPlan[State]
macros, but a few places didn't do it like that. For consistency
and readability, convert all the stragglers to use the macros.
(See also commit 40f42d2a3, which did some similar cleanup a few
years ago, but missed these cases.)
Richard Guo
Discussion: https://postgr.es/m/CAMbWs4-vYhh1xsa_veah4PUed2Xq=Ed_YH3=Mqt5A3Y=EgfCEg@mail.gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/88210542106de5b26fe6aa088d1811b68502d224
Modified Files
--------------
src/backend/executor/execAmi.c | 10 +++++-----
src/backend/executor/execCurrent.c | 2 +-
src/backend/executor/nodeAgg.c | 2 +-
src/backend/executor/nodeGather.c | 4 ++--
src/backend/executor/nodeGatherMerge.c | 4 ++--
src/backend/executor/nodeHash.c | 6 ++++--
src/backend/executor/nodeHashjoin.c | 15 +++++++++------
src/backend/executor/nodeLimit.c | 6 ++++--
src/backend/executor/nodeLockRows.c | 6 ++++--
src/backend/executor/nodeMergejoin.c | 11 +++++++----
src/backend/executor/nodeProjectSet.c | 6 ++++--
src/backend/executor/nodeResult.c | 7 ++++---
src/backend/executor/nodeSetOp.c | 8 +++++---
src/backend/executor/nodeUnique.c | 6 ++++--
14 files changed, 56 insertions(+), 37 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2022-07-07 21:46:12 | pgsql: Only allow returning string types or bytea from json_serialize |
Previous Message | Fujii Masao | 2022-07-07 13:41:37 | pgsql: Add checkpoint and REDO LSN to log_checkpoints message. |