summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarko Mäkelä <marko.makela@mariadb.com>2021-03-10 11:08:51 +0200
committerMarko Mäkelä <marko.makela@mariadb.com>2021-03-10 11:08:51 +0200
commit1af855819300ba4e4dd3d582c5c521ff99321152 (patch)
treeeadd727751aeae178c4165bd0dbc7c25fe7a3601
parentee12b055ffdf2d7570be792f82f23c15cf71a0e4 (diff)
downloadmariadb-git-1af855819300ba4e4dd3d582c5c521ff99321152.tar.gz
MDEV-25101 Assertion !strcmp(field->name, "table_name") failed
btr_node_ptr_max_size(): Let us remove the debug assertion that was added in MDEV-14637. The assertion assumed that no additional indexes exist in mysql.innodb_index_stats or mysql.innodb_table_stats. The code path is working around an incorrect definition of a table, interpreting VARCHAR(64) as the more correct VARCHAR(199). No test case will be added, because MDEV-24579 proves that executing DDL on the statistics tables involves a race condition. The test case included the following: ALTER TABLE mysql.innodb_index_stats ADD KEY (stat_name); CREATE TABLE t (a INT) ENGINE=InnoDB STATS_PERSISTENT=1;
-rw-r--r--storage/innobase/btr/btr0cur.cc1
1 files changed, 0 insertions, 1 deletions
diff --git a/storage/innobase/btr/btr0cur.cc b/storage/innobase/btr/btr0cur.cc
index 3d03c55bf15..8f2acdb35bd 100644
--- a/storage/innobase/btr/btr0cur.cc
+++ b/storage/innobase/btr/btr0cur.cc
@@ -837,7 +837,6 @@ static ulint btr_node_ptr_max_size(const dict_index_t* index)
TABLE_STATS_NAME)
|| !strcmp(index->table->name.m_name,
INDEX_STATS_NAME))) {
- ut_ad(!strcmp(field->name, "table_name"));
/* Interpret "table_name" as VARCHAR(199) even
if it was incorrectly defined as VARCHAR(64).
While the caller of ha_innobase enforces the