diff options
| author | Sergei Golubchik <serg@mariadb.org> | 2018-04-20 10:10:33 +0200 |
|---|---|---|
| committer | Sergei Golubchik <serg@mariadb.org> | 2018-04-20 10:24:44 +0200 |
| commit | bcb36ee21e2515d17dcf03b760487e64eb780f2b (patch) | |
| tree | e951c3fc19c732b9f88dac31ff06254ebffb2778 /sql/sql_partition_admin.cc | |
| parent | 86718fda4e5363b05aa2beaac596ec614aaa7a0f (diff) | |
| download | mariadb-git-bcb36ee21e2515d17dcf03b760487e64eb780f2b.tar.gz | |
MDEV-15456 Server crashes upon adding or dropping a partition in ALTER under LOCK TABLE after ER_SAME_NAME_PARTITION
ALTER TABLE ... ADD PARTITION modifies the open TABLE structure,
and sets table->need_reopen=1 to reset these modifications
in case of an error.
But under LOCK TABLES the table isn't get reopened, despite need_reopen.
Fixed by reopening need_reopen tables under LOCK TABLE.
Diffstat (limited to 'sql/sql_partition_admin.cc')
| -rw-r--r-- | sql/sql_partition_admin.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/sql_partition_admin.cc b/sql/sql_partition_admin.cc index 9b471f97521..39b7f4c49f6 100644 --- a/sql/sql_partition_admin.cc +++ b/sql/sql_partition_admin.cc @@ -630,7 +630,7 @@ bool Sql_cmd_alter_table_exchange_partition:: better to keep master/slave in consistent state. Alternative would be to try to revert the exchange operation and issue error. */ - (void) thd->locked_tables_list.reopen_tables(thd); + (void) thd->locked_tables_list.reopen_tables(thd, false); if ((error= write_bin_log(thd, TRUE, thd->query(), thd->query_length()))) { |