summaryrefslogtreecommitdiff
path: root/mysql-test/main/mysqld--help.test
diff options
context:
space:
mode:
authorMonty <monty@mariadb.org>2022-05-31 17:36:32 +0300
committerOleg Smirnov <olernov@gmail.com>2022-07-26 22:27:28 +0700
commitb3c74bdc1f09b2b8babd7f2bd9d52df2749ddcc3 (patch)
tree9e3364bb4ef289d8ffd951499a321f7d126119a9 /mysql-test/main/mysqld--help.test
parent213772f99a109d3ea58fb40c5c48422d458366fa (diff)
downloadmariadb-git-b3c74bdc1f09b2b8babd7f2bd9d52df2749ddcc3.tar.gz
Improve pruning in greedy_search by sorting tables during search
MDEV-28073 Slow query performance in MariaDB when using many tables The faster we can find a good query plan, the more options we have for finding and pruning (ignoring) bad plans. This patch adds sorting of plans to best_extension_by_limited_search(). The plans, from best_access_path() are sorted according to the numbers of found rows. This allows us to faster find 'good tables' and we are thus able to eliminate 'bad plans' faster. One side effect of this patch is that if two tables have equal cost, the table that which was used earlier in the query is preferred. This allows users to improve plans by reordering eq_ref tables in the order they would like them to be uses. Result changes caused by the patch: - Traces are different as now we print the cost for using tables before we start considering them in the plan. - Table order are changed for some plans. In most cases this is because the plans are equal and tables are in this case sorted according to their usage in the original query. - A few plans was changed as the optimizer was able to find a better plan (that was pruned by the original code). Other things: - Added a new statistic variable: "optimizer_join_prefixes_check_calls", which counts number of calls to best_extension_by_limited_search(). This can be used to check the prune efficiency in greedy_search(). - Added variable "JOIN_TAB::embedded_dependent" to be able to handle XX IN (SELECT..) in the greedy_optimizer. The idea is that we should prune a table if any of the tables in embedded_dependent is not yet read. - When using many tables in a query, there will be some additional memory usage as we need to pre-allocate table of table_count*table_count*sizeof(POSITION) objects (POSITION is 312 bytes for now) to hold the pre-calculated best_access_path() information. This memory usage is offset by the expected performance improvement when using many tables in a query. - Removed the code from an earlier patch to keep the table order in join->best_ref in the original order. This is not needed anymore as we are now sorting the tables for each best_extension_by_limited_search() call.
Diffstat (limited to 'mysql-test/main/mysqld--help.test')
-rw-r--r--mysql-test/main/mysqld--help.test3
1 files changed, 2 insertions, 1 deletions
diff --git a/mysql-test/main/mysqld--help.test b/mysql-test/main/mysqld--help.test
index 95ae49b5c89..ab2a61982da 100644
--- a/mysql-test/main/mysqld--help.test
+++ b/mysql-test/main/mysqld--help.test
@@ -38,7 +38,8 @@ perl;
test-sql-discovery query-cache-info password-reuse-check
query-response-time metadata-lock-info locales unix-socket
wsrep file-key-management cracklib-password-check user-variables
- thread-pool-groups thread-pool-queues thread-pool-stats thread-pool-waits/;
+ thread-pool-groups thread-pool-queues thread-pool-stats
+ thread-pool-waits hashicorp provider/;
# And substitute the content some environment variables with their
# names: