diff options
| author | Sergei Petrunia <sergey@mariadb.com> | 2022-04-04 12:32:22 +0300 |
|---|---|---|
| committer | Sergei Petrunia <sergey@mariadb.com> | 2022-04-29 10:48:26 +0300 |
| commit | 3f68c2169e16814e8e268d0f9a9f29aee522453e (patch) | |
| tree | c510410cc65f3943c4ed69fd7e93ee9804cc9cb5 /sql/sql_prepare.cc | |
| parent | 02c3babdec0c931c6983863bff129114f750bd22 (diff) | |
| download | mariadb-git-3f68c2169e16814e8e268d0f9a9f29aee522453e.tar.gz | |
MDEV-28201: Server crashes upon SHOW ANALYZE/EXPLAIN FORMAT=JSON
- Describe the lifetime of EXPLAIN data structures in
sql_explain.h:ExplainDataStructureLifetime.
- Make Item_field::set_field() call set_refers_to_temp_table()
when it refers to a temp. table.
- Introduce QT_DONT_ACCESS_TMP_TABLES flag for Item::print.
It directs Item_field::print to not try access its the
temp table.
- Introduce Explain_query::notify_tables_are_closed()
and call it right before the query closes its tables.
- Make Explain data stuctures' print_explain_json() methods
accept "no_tmp_tbl" parameter which means pass
QT_DONT_ACCESS_TMP_TABLES when printing items.
- Make Show_explain_request::call_in_target_thread() not call
set_current_thd(). This wasn't needed as the code inside
lex->print_explain() uses output->thd anyway. output->thd
refers to the SHOW command's THD object.
Diffstat (limited to 'sql/sql_prepare.cc')
| -rw-r--r-- | sql/sql_prepare.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/sql_prepare.cc b/sql/sql_prepare.cc index 83c064c63c6..b10a81d5e56 100644 --- a/sql/sql_prepare.cc +++ b/sql/sql_prepare.cc @@ -4444,7 +4444,7 @@ bool Prepared_statement::prepare(const char *packet, uint packet_len) /* No need to commit statement transaction, it's not started. */ DBUG_ASSERT(thd->transaction->stmt.is_empty()); - close_thread_tables(thd); + close_thread_tables_for_query(thd); thd->mdl_context.rollback_to_savepoint(mdl_savepoint); /* |