diff options
| author | Heikki Linnakangas | 2020-11-24 08:45:00 +0000 |
|---|---|---|
| committer | Heikki Linnakangas | 2020-11-24 08:45:00 +0000 |
| commit | 0a2bc5d61e713e3fe72438f020eea5fcc90b0f0b (patch) | |
| tree | 8f630f828fa6bbfb5984f037118211cd68d8a50e /src/include/optimizer | |
| parent | e522024bd8dd28a0f13dcccfd39170698f45c939 (diff) | |
Move per-agg and per-trans duplicate finding to the planner.
This has the advantage that the cost estimates for aggregates can count
the number of calls to transition and final functions correctly.
Bump catalog version, because views can contain Aggrefs.
Reviewed-by: Andres Freund
Discussion: https://www.postgresql.org/message-id/b2e3536b-1dbc-8303-c97e-89cb0b4a9a48%40iki.fi
Diffstat (limited to 'src/include/optimizer')
| -rw-r--r-- | src/include/optimizer/clauses.h | 2 | ||||
| -rw-r--r-- | src/include/optimizer/prep.h | 8 |
2 files changed, 8 insertions, 2 deletions
diff --git a/src/include/optimizer/clauses.h b/src/include/optimizer/clauses.h index 2584ffc72fc..68855d0cee7 100644 --- a/src/include/optimizer/clauses.h +++ b/src/include/optimizer/clauses.h @@ -24,8 +24,6 @@ typedef struct } WindowFuncLists; extern bool contain_agg_clause(Node *clause); -extern void get_agg_clause_costs(PlannerInfo *root, Node *clause, - AggSplit aggsplit, AggClauseCosts *costs); extern bool contain_window_function(Node *clause); extern WindowFuncLists *find_window_functions(Node *clause, Index maxWinRef); diff --git a/src/include/optimizer/prep.h b/src/include/optimizer/prep.h index 19c92302b03..0abe6bec00e 100644 --- a/src/include/optimizer/prep.h +++ b/src/include/optimizer/prep.h @@ -39,8 +39,16 @@ extern List *preprocess_targetlist(PlannerInfo *root); extern PlanRowMark *get_plan_rowmark(List *rowmarks, Index rtindex); /* + * prototypes for prepagg.c + */ +extern void get_agg_clause_costs(PlannerInfo *root, AggSplit aggsplit, + AggClauseCosts *agg_costs); +extern void preprocess_aggrefs(PlannerInfo *root, Node *clause); + +/* * prototypes for prepunion.c */ extern RelOptInfo *plan_set_operations(PlannerInfo *root); + #endif /* PREP_H */ |