database/mariadb
1
0
Fork 0
mirror of https://github.com/MariaDB/server.git synced 2025-08-07 00:04:31 +03:00
Code Activity

MDEV-17726: A better fix

Browse Source 
THD::close_temporary_tables(): Revert the change.

ha_innobase::delete_table(): Move the work-around inside
a debug assertion, and check thd_kill_level() instead of thd_killed(),
because the latter would not hold for KILL_CONNECTION.
This commit is contained in:
Marko Mäkelä
2018-11-16 10:36:57 +02:00
parent 705abdebaf
commit ab812c1089
2 changed files with 18 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
sql/temporary_tables.cc
View File

@@ -1,5 +1,5 @@
/* /*
Copyright (c) 2016,2018 MariaDB Corporation Copyright (c) 2016 MariaDB Corporation
This program is free software; you can redistribute it and/or modify This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by it under the terms of the GNU General Public License as published by
@@ -476,7 +476,6 @@ bool THD::close_temporary_tables()
} }
DBUG_ASSERT(!rgi_slave); DBUG_ASSERT(!rgi_slave);
lex->sql_command = SQLCOM_DROP_TABLE;
/* /*
Ensure we don't have open HANDLERs for tables we are about to close. Ensure we don't have open HANDLERs for tables we are about to close.

26
storage/innobase/handler/ha_innodb.cc
View File

@@ -13187,17 +13187,25 @@ inline int ha_innobase::delete_table(const char* name, enum_sql_command sqlcom)
int ha_innobase::delete_table(const char* name) int ha_innobase::delete_table(const char* name)
{ {
enum_sql_command sqlcom = enum_sql_command(thd_sql_command(ha_thd())); enum_sql_command sqlcom = enum_sql_command(thd_sql_command(ha_thd()));
/* SQLCOM_TRUNCATE should be passed via ha_innobase::truncate() only.
if (sqlcom == SQLCOM_TRUNCATE On client disconnect, when dropping temporary tables, the
&& thd_killed(ha_thd()) previous sqlcom would not be overwritten. In such a case, we
&& (m_prebuilt == NULL will have thd_kill_level() != NOT_KILLED, !m_prebuilt can
|| dict_table_is_temporary(m_prebuilt->table))) { hold, and sqlcom could be anything, including TRUNCATE.
sqlcom = SQLCOM_DROP_TABLE;
}
/* SQLCOM_TRUNCATE will be passed via ha_innobase::truncate() only. */ The sqlcom only matters for persistent tables; no persistent
DBUG_ASSERT(sqlcom != SQLCOM_TRUNCATE); metadata or FOREIGN KEY metadata is kept for temporary
return delete_table(name, sqlcom); tables. Therefore, we relax the assertion. If there is a bug
that slips through this assertion due to !m_prebuilt, the
worst impact should be that on DROP TABLE of a persistent
table, FOREIGN KEY constraints will be ignored and their
metadata will not be removed. */
DBUG_ASSERT(sqlcom != SQLCOM_TRUNCATE
|| (thd_kill_level(ha_thd()) != NOT_KILLED
&& (!m_prebuilt
|| m_prebuilt->table->is_temporary())));
return delete_table(name, sqlcom);
} }
/** Remove all tables in the named database inside InnoDB. /** Remove all tables in the named database inside InnoDB.