Skip to content

Commit a48e63c

Browse files
committed
Fix compile error and test failure:
- Don't use 'res' uninitialized - multiply it by col_non_nulls before set_if_bigger(...) call.
1 parent 096995b commit a48e63c

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

sql/sql_statistics.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4369,8 +4369,8 @@ double get_column_range_cardinality(Field *field,
43694369
if (hist && hist->is_usable(thd))
43704370
{
43714371
sel= hist->range_selectivity(field, min_endp, max_endp);
4372-
set_if_bigger(res, col_stats->get_avg_frequency());
4373-
} else
4372+
}
4373+
else
43744374
{
43754375
double min_mp_pos, max_mp_pos;
43764376
if (min_endp && !(field->null_ptr && min_endp->key[0]))
@@ -4395,6 +4395,7 @@ double get_column_range_cardinality(Field *field,
43954395
sel = (max_mp_pos - min_mp_pos);
43964396
}
43974397
res= col_non_nulls * sel;
4398+
set_if_bigger(res, col_stats->get_avg_frequency());
43984399
}
43994400
else
44004401
res= col_non_nulls;

0 commit comments

Comments
 (0)