diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2024-04-23 12:55:26 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2024-04-23 12:55:26 -0400 |
| commit | b7d35d393edbe2d4333dde81496e8a362abc85bd (patch) | |
| tree | 72c4dfd40d1b1d422e1b7b0493ed698df63a16e8 /src/backend/executor/execExpr.c | |
| parent | 598e0114a3b16bb7edcaba6784b3794f816889ae (diff) | |
| download | postgresql-b7d35d393edbe2d4333dde81496e8a362abc85bd.tar.gz postgresql-b7d35d393edbe2d4333dde81496e8a362abc85bd.zip | |
Remove some unnecessary fields from executor nodes.
JsonExprState.input_finfo is only assigned to, never read, and
it's really fairly useless since the value can be gotten out of
the adjacent input_fcinfo field. Let's remove it before someone
starts to depend on it.
While here, also remove TidScanState.tss_htup and AggState.combinedproj,
which are referenced nowhere. Those should have been removed by the
commits that caused them to become disused, but were not.
I don't think a catversion bump is necessary here, since plan trees
are never stored on disk.
Matthias van de Meent
Discussion: https://postgr.es/m/CAEze2WjsY4d0TBymLNGK4zpttUcg_YZaTjyWz2VfDUV6YH8wXQ@mail.gmail.com
Diffstat (limited to 'src/backend/executor/execExpr.c')
| -rw-r--r-- | src/backend/executor/execExpr.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/backend/executor/execExpr.c b/src/backend/executor/execExpr.c index eb5ac208248..b9ebc827a74 100644 --- a/src/backend/executor/execExpr.c +++ b/src/backend/executor/execExpr.c @@ -4393,7 +4393,6 @@ ExecInitJsonExpr(JsonExpr *jsexpr, ExprState *state, fcinfo->args[2].isnull = false; fcinfo->context = (Node *) escontext; - jsestate->input_finfo = finfo; jsestate->input_fcinfo = fcinfo; } |