diff options
author | Marko Mäkelä <marko.makela@mariadb.com> | 2020-07-27 19:33:21 +0300 |
---|---|---|
committer | Marko Mäkelä <marko.makela@mariadb.com> | 2020-07-28 14:02:11 +0300 |
commit | cf3c3cce1d1078d6ac2c0d74a3cd0ff444c63db9 (patch) | |
tree | adeacf81fa3ce0986a5351c70bcf7ad93e2fba06 | |
parent | 2c5831b2c5348edf19a4c3c91ebed21c6e8412b9 (diff) | |
download | mariadb-git-cf3c3cce1d1078d6ac2c0d74a3cd0ff444c63db9.tar.gz |
MDEV-23304 Insert into TEMPORARY TABLE fails to invoke mtr_t::set_modified()
In MDEV-21724 or possibly already in MDEV-12353 a bug was introduced to
the handling of temporary tables.
Whether or not redo log will be written, mtr_t::set_modified()
must be invoked to register any changes to a page.
page_cur_insert_rec_low(): Invoke mtr_t::set_modified() also
when skipping the redo log write.
-rw-r--r-- | storage/innobase/page/page0cur.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/storage/innobase/page/page0cur.cc b/storage/innobase/page/page0cur.cc index 36fdde82cb3..5f42deaaf8e 100644 --- a/storage/innobase/page/page0cur.cc +++ b/storage/innobase/page/page0cur.cc @@ -1493,7 +1493,10 @@ inc_dir: rec_set_bit_field_1(next_rec, n_owned + 1, REC_NEW_N_OWNED, REC_N_OWNED_MASK, REC_N_OWNED_SHIFT); if (mtr->get_log_mode() != MTR_LOG_ALL) + { + mtr->set_modified(*block); goto copied; + } const byte * const c_start= cur->rec - extra_size; if (extra_size > REC_N_NEW_EXTRA_BYTES && @@ -1532,7 +1535,10 @@ inc_dir: rec_set_bit_field_1(next_rec, n_owned + 1, REC_OLD_N_OWNED, REC_N_OWNED_MASK, REC_N_OWNED_SHIFT); if (mtr->get_log_mode() != MTR_LOG_ALL) + { + mtr->set_modified(*block); goto copied; + } ut_ad(extra_size > REC_N_OLD_EXTRA_BYTES); const byte * const c_start= cur->rec - extra_size; |