summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarko Mäkelä <marko.makela@mariadb.com>2018-02-04 13:11:49 +0200
committerMarko Mäkelä <marko.makela@mariadb.com>2018-02-04 13:11:49 +0200
commitd6ed077fc82c2f4d20895db9a6b16ca295f23c33 (patch)
treee87770a5def3942cbe5aee2c325ab62921560ba4
parentc8299e6278e42bdd2540e75305ba4effb7259f6e (diff)
downloadmariadb-git-d6ed077fc82c2f4d20895db9a6b16ca295f23c33.tar.gz
Clarify a comment after MDEV-15061
-rw-r--r--storage/innobase/lock/lock0lock.cc7
1 files changed, 5 insertions, 2 deletions
diff --git a/storage/innobase/lock/lock0lock.cc b/storage/innobase/lock/lock0lock.cc
index d8d01f88a05..3d66e50cce9 100644
--- a/storage/innobase/lock/lock0lock.cc
+++ b/storage/innobase/lock/lock0lock.cc
@@ -5853,8 +5853,11 @@ lock_rec_block_validate(
buf_block_t* block;
mtr_t mtr;
- /* Make sure that the tablespace is not deleted while we are
- trying to access the page. */
+ /* Transactional locks should never refer to dropped
+ tablespaces, because all DDL operations that would drop or
+ discard or rebuild a tablespace do hold an exclusive table
+ lock, which would conflict with any locks referring to the
+ tablespace from other transactions. */
if (fil_space_t* space = fil_space_acquire(space_id)) {
dberr_t err = DB_SUCCESS;
mtr_start(&mtr);