diff options
| author | Andres Freund | 2020-02-07 03:06:16 +0000 |
|---|---|---|
| committer | Andres Freund | 2020-02-07 03:54:43 +0000 |
| commit | 1fdb7f9789c4550204cd62d1746a7deed1dc4c29 (patch) | |
| tree | 877c68ff731bf42a811be492dca9c79db71494be /src/include/executor | |
| parent | 1ec7679f1b67e84be688a311dce234eeaa1d5de8 (diff) | |
expression eval: Don't redundantly keep track of AggState.
It's already tracked via ExprState->parent, so we don't need to also
include it in ExprEvalStep. When that code originally was written
ExprState->parent didn't exist, but it since has been introduced in
6719b238e8f.
Author: Andres Freund
Discussion: https://postgr.es/m/[email protected]
Diffstat (limited to 'src/include/executor')
| -rw-r--r-- | src/include/executor/execExpr.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/include/executor/execExpr.h b/src/include/executor/execExpr.h index 7112558363f..73a2ca8c6dd 100644 --- a/src/include/executor/execExpr.h +++ b/src/include/executor/execExpr.h @@ -569,7 +569,6 @@ typedef struct ExprEvalStep /* for EEOP_GROUPING_FUNC */ struct { - AggState *parent; /* parent Agg */ List *clauses; /* integer list of column numbers */ } grouping_func; @@ -597,7 +596,6 @@ typedef struct ExprEvalStep /* for EEOP_AGG_*DESERIALIZE */ struct { - AggState *aggstate; FunctionCallInfo fcinfo_data; int jumpnull; } agg_deserialize; @@ -625,7 +623,6 @@ typedef struct ExprEvalStep /* for EEOP_AGG_INIT_TRANS */ struct { - AggState *aggstate; AggStatePerTrans pertrans; ExprContext *aggcontext; int setno; @@ -637,7 +634,6 @@ typedef struct ExprEvalStep /* for EEOP_AGG_STRICT_TRANS_CHECK */ struct { - AggState *aggstate; int setno; int transno; int setoff; @@ -647,7 +643,6 @@ typedef struct ExprEvalStep /* for EEOP_AGG_{PLAIN,ORDERED}_TRANS* */ struct { - AggState *aggstate; AggStatePerTrans pertrans; ExprContext *aggcontext; int setno; |