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 2584ffc72f..68855d0cee 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 19c92302b0..0abe6bec00 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 */ |