diff options
| author | Amit Langote <amitlan@postgresql.org> | 2023-09-28 09:44:39 +0900 |
|---|---|---|
| committer | Amit Langote <amitlan@postgresql.org> | 2023-09-28 09:44:39 +0900 |
| commit | d060e921ea5aa47b6265174c32e1128cebdbc3df (patch) | |
| tree | c99915a37571d3e5e1792069677069d4bc1d9c86 /src/backend/executor/nodeSubqueryscan.c | |
| parent | 9210afd3bcd65feccb883ace4ed6dcef6a684585 (diff) | |
| download | postgresql-d060e921ea5aa47b6265174c32e1128cebdbc3df.tar.gz postgresql-d060e921ea5aa47b6265174c32e1128cebdbc3df.zip | |
Remove obsolete executor cleanup code
This commit removes unnecessary ExecExprFreeContext() calls in
ExecEnd* routines because the actual cleanup is managed by
FreeExecutorState(). With no callers remaining for
ExecExprFreeContext(), this commit also removes the function.
This commit also drops redundant ExecClearTuple() calls, because
ExecResetTupleTable() in ExecEndPlan() already takes care of
resetting and dropping all TupleTableSlots initialized with
ExecInitScanTupleSlot() and ExecInitExtraTupleSlot().
After these modifications, the ExecEnd*() routines for ValuesScan,
NamedTuplestoreScan, and WorkTableScan became redundant. So, this
commit removes them.
Reviewed-by: Robert Haas
Discussion: https://postgr.es/m/CA+HiwqFGkMSge6TgC9KQzde0ohpAycLQuV7ooitEEpbKB0O_mg@mail.gmail.com
Diffstat (limited to 'src/backend/executor/nodeSubqueryscan.c')
| -rw-r--r-- | src/backend/executor/nodeSubqueryscan.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/src/backend/executor/nodeSubqueryscan.c b/src/backend/executor/nodeSubqueryscan.c index 42471bfc041..1ee62956601 100644 --- a/src/backend/executor/nodeSubqueryscan.c +++ b/src/backend/executor/nodeSubqueryscan.c @@ -168,18 +168,6 @@ void ExecEndSubqueryScan(SubqueryScanState *node) { /* - * Free the exprcontext - */ - ExecFreeExprContext(&node->ss.ps); - - /* - * clean out the upper tuple table - */ - if (node->ss.ps.ps_ResultTupleSlot) - ExecClearTuple(node->ss.ps.ps_ResultTupleSlot); - ExecClearTuple(node->ss.ss_ScanTupleSlot); - - /* * close down subquery */ ExecEndNode(node->subplan); |