diff options
author | unknown <timour@askmonty.org> | 2012-06-06 22:26:40 +0300 |
---|---|---|
committer | unknown <timour@askmonty.org> | 2012-06-06 22:26:40 +0300 |
commit | c2677de7aca09a0ba4b680b5227bda3865ab9290 (patch) | |
tree | 980157e9eaf1062641572da371d1ae7fcc371625 /sql/item_subselect.cc | |
parent | 8efc63ba5d32b77501226921ee503b9ae513a365 (diff) | |
parent | 7ddd5418d01e60dba2ae69a668e7c9f811613451 (diff) | |
download | mariadb-git-c2677de7aca09a0ba4b680b5227bda3865ab9290.tar.gz |
Merge the fix for lp:944706, mdev-193
Diffstat (limited to 'sql/item_subselect.cc')
-rw-r--r-- | sql/item_subselect.cc | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/sql/item_subselect.cc b/sql/item_subselect.cc index 5458a2fb968..bbd3bf35a2d 100644 --- a/sql/item_subselect.cc +++ b/sql/item_subselect.cc @@ -523,6 +523,48 @@ void Item_subselect::recalc_used_tables(st_select_lex *new_parent, */ } + +/** + Determine if a subquery is expensive to execute during query optimization. + + @details The cost of execution of a subquery is estimated based on an + estimate of the number of rows the subquery will access during execution. + This measure is used instead of JOIN::read_time, because it is considered + to be much more reliable than the cost estimate. + + @return true if the subquery is expensive + @return false otherwise +*/ +bool Item_subselect::is_expensive() +{ + double examined_rows= 0; + + for (SELECT_LEX *sl= unit->first_select(); sl; sl= sl->next_select()) + { + JOIN *cur_join= sl->join; + if (!cur_join) + continue; + + /* If a subquery is not optimized we cannot estimate its cost. */ + if (!cur_join->join_tab) + return true; + + if (sl->first_inner_unit()) + { + /* + Subqueries that contain subqueries are considered expensive. + @todo: accumulate the cost of subqueries. + */ + return true; + } + + examined_rows+= cur_join->get_examined_rows(); + } + + return (examined_rows > thd->variables.expensive_subquery_limit); +} + + bool Item_subselect::walk(Item_processor processor, bool walk_subquery, uchar *argument) { |