diff options
author | Sergei Petrunia <psergey@askmonty.org> | 2019-10-04 20:16:39 +0300 |
---|---|---|
committer | Sergei Petrunia <psergey@askmonty.org> | 2019-10-04 20:18:31 +0300 |
commit | 37570e845bc5eb20e277340015c5fd36feb7a823 (patch) | |
tree | 89683ea47820f919f7a7fe6721747970e3910823 /sql/opt_range_mrr.cc | |
parent | a5c34bc200eb38791fdba170968b72bf9139789e (diff) | |
download | mariadb-git-37570e845bc5eb20e277340015c5fd36feb7a823.tar.gz |
MDEV-20740: Odd computations in calculate_cond_selectivity_for_table
Make SEL_ARG graph traversal code in sel_arg_range_seq_next() set
max_key_parts first.
(Pushing to 10.4 first)
Diffstat (limited to 'sql/opt_range_mrr.cc')
-rw-r--r-- | sql/opt_range_mrr.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sql/opt_range_mrr.cc b/sql/opt_range_mrr.cc index d3a1e155fb7..4afa06a7ca0 100644 --- a/sql/opt_range_mrr.cc +++ b/sql/opt_range_mrr.cc @@ -247,6 +247,7 @@ walk_up_n_right: uint min_key_length= (uint)(cur->min_key - seq->param->min_key); range->ptr= (char*)(intptr)(key_tree->part); + uint max_key_parts; if (cur->min_key_flag & GEOM_FLAG) { range->range_flag= cur->min_key_flag; @@ -256,9 +257,11 @@ walk_up_n_right: range->start_key.length= min_key_length; range->start_key.keypart_map= make_prev_keypart_map(cur->min_key_parts); range->start_key.flag= (ha_rkey_function) (cur->min_key_flag ^ GEOM_FLAG); + max_key_parts= cur->min_key_parts; } else { + max_key_parts= MY_MAX(cur->min_key_parts, cur->max_key_parts); range->range_flag= cur->min_key_flag | cur->max_key_flag; range->start_key.key= seq->param->min_key; @@ -336,7 +339,7 @@ walk_up_n_right: } } seq->param->range_count++; - seq->param->max_key_part=MY_MAX(seq->param->max_key_part,key_tree->part); + seq->param->max_key_parts= MY_MAX(seq->param->max_key_parts, max_key_parts); return 0; } |