Skip to content

Commit

Permalink
Fix missing cost initialization
Browse files Browse the repository at this point in the history
The Sanitizer build found a missing memory initialization of the
aggregation costs. This PR adds the missing initialization.
  • Loading branch information
jnidzwetzki committed Sep 15, 2023
1 parent 98507d4 commit c802c31
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/planner/partialize.c
Original file line number Diff line number Diff line change
Expand Up @@ -767,6 +767,10 @@ ts_pushdown_partial_agg(PlannerInfo *root, Hypertable *ht, RelOptInfo *input_rel
/* Calculate aggregation costs */
if (!extra_data->partial_costs_set)
{
/* Init costs */
MemSet(&extra_data->agg_partial_costs, 0, sizeof(AggClauseCosts));
MemSet(&extra_data->agg_final_costs, 0, sizeof(AggClauseCosts));

/* partial phase */
get_agg_clause_costs_compat(root,
(Node *) partial_grouping_target->exprs,
Expand Down

0 comments on commit c802c31

Please sign in to comment.