summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarko Mäkelä <marko.makela@mariadb.com>2021-03-04 18:11:25 +0200
committerMarko Mäkelä <marko.makela@mariadb.com>2021-03-04 18:11:25 +0200
commit7759991a06d54630214f19eaa0ec39bd21bf09df (patch)
treeb37670a9232160c94e6075b7970a80bc9466a0e9
parent978e48c96cf22b1f17444dd54b6b64b28752881e (diff)
downloadmariadb-git-7759991a06d54630214f19eaa0ec39bd21bf09df.tar.gz
fixup 58b56f14a096285a0e51b2233fc35398f1b01f5a: Remove dead code
row_prebuilt_t::m_no_prefetch: Remove (it was always false). row_prebuilt_t::m_read_virtual_key: Remove (it was always false). Only ha_innopart ever set these fields.
-rw-r--r--storage/innobase/include/row0mysql.h8
-rw-r--r--storage/innobase/row/row0mysql.cc5
-rw-r--r--storage/innobase/row/row0sel.cc26
3 files changed, 6 insertions, 33 deletions
diff --git a/storage/innobase/include/row0mysql.h b/storage/innobase/include/row0mysql.h
index d5cdd4a3f17..8738f991368 100644
--- a/storage/innobase/include/row0mysql.h
+++ b/storage/innobase/include/row0mysql.h
@@ -1,7 +1,7 @@
/*****************************************************************************
Copyright (c) 2000, 2017, Oracle and/or its affiliates. All Rights Reserved.
-Copyright (c) 2017, 2019, MariaDB Corporation.
+Copyright (c) 2017, 2021, MariaDB Corporation.
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 the Free Software
@@ -784,12 +784,6 @@ struct row_prebuilt_t {
search key values from MySQL format
to InnoDB format.*/
uint srch_key_val_len; /*!< Size of search key */
- /** Disable prefetch. */
- bool m_no_prefetch;
-
- /** Return materialized key for secondary index scan */
- bool m_read_virtual_key;
-
/** The MySQL table object */
TABLE* m_mysql_table;
};
diff --git a/storage/innobase/row/row0mysql.cc b/storage/innobase/row/row0mysql.cc
index 312c35c58a3..c2f9186d408 100644
--- a/storage/innobase/row/row0mysql.cc
+++ b/storage/innobase/row/row0mysql.cc
@@ -1,7 +1,7 @@
/*****************************************************************************
Copyright (c) 2000, 2018, Oracle and/or its affiliates. All Rights Reserved.
-Copyright (c) 2015, 2020, MariaDB Corporation.
+Copyright (c) 2015, 2021, MariaDB Corporation.
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 the Free Software
@@ -958,9 +958,6 @@ row_create_prebuilt(
prebuilt->fts_doc_id_in_read_set = 0;
prebuilt->blob_heap = NULL;
- prebuilt->m_no_prefetch = false;
- prebuilt->m_read_virtual_key = false;
-
DBUG_RETURN(prebuilt);
}
diff --git a/storage/innobase/row/row0sel.cc b/storage/innobase/row/row0sel.cc
index 0451a240ffc..864f7bd54ab 100644
--- a/storage/innobase/row/row0sel.cc
+++ b/storage/innobase/row/row0sel.cc
@@ -2,7 +2,7 @@
Copyright (c) 1997, 2017, Oracle and/or its affiliates. All Rights Reserved.
Copyright (c) 2008, Google Inc.
-Copyright (c) 2015, 2020, MariaDB Corporation.
+Copyright (c) 2015, 2021, MariaDB Corporation.
Portions of this file contain modifications contributed and copyrighted by
Google, Inc. Those modifications are gratefully acknowledged and are described
@@ -3168,8 +3168,7 @@ static bool row_sel_store_mysql_rec(
search or virtual key read is not requested. */
if (!rec_clust
|| !prebuilt->index->has_virtual()
- || (!prebuilt->read_just_key
- && !prebuilt->m_read_virtual_key)) {
+ || !prebuilt->read_just_key) {
/* Initialize the NULL bit. */
mysql_rec[templ->mysql_null_byte_offset]
|= (byte) templ->mysql_null_bit_mask;
@@ -3185,23 +3184,8 @@ static bool row_sel_store_mysql_rec(
const dfield_t* dfield = dtuple_get_nth_v_field(
vrow, col->v_pos);
- /* If this is a partitioned table, it might request
- InnoDB to fill out virtual column data for serach
- index key values while other non key columns are also
- getting selected. The non-key virtual columns may
- not be materialized and we should skip them. */
if (dfield_get_type(dfield)->mtype == DATA_MISSING) {
-#ifdef UNIV_DEBUG
- ulint prefix;
-#endif /* UNIV_DEBUG */
- ut_ad(prebuilt->m_read_virtual_key);
-
- /* If it is part of index key the data should
- have been materialized. */
- ut_ad(dict_index_get_nth_col_or_prefix_pos(
- prebuilt->index, col->v_pos, false,
- true, &prefix) == ULINT_UNDEFINED);
-
+ ut_ad("no ha_innopart in MariaDB" == 0);
continue;
}
@@ -4377,8 +4361,7 @@ row_search_mvcc(
index key, if this is covered index scan or virtual key read is
requested. */
bool need_vrow = dict_index_has_virtual(prebuilt->index)
- && (prebuilt->read_just_key
- || prebuilt->m_read_virtual_key);
+ && prebuilt->read_just_key;
/* Reset the new record lock info if srv_locks_unsafe_for_binlog
is set or session is using a READ COMMITED isolation level. Then
@@ -5524,7 +5507,6 @@ use_covering_index:
if ((match_mode == ROW_SEL_EXACT
|| prebuilt->n_rows_fetched >= MYSQL_FETCH_CACHE_THRESHOLD)
&& prebuilt->select_lock_type == LOCK_NONE
- && !prebuilt->m_no_prefetch
&& !prebuilt->templ_contains_blob
&& !prebuilt->clust_index_was_generated
&& !prebuilt->used_in_HANDLER