Skip to content

Commit 2770e80

Browse files
committed
MariaRocks: remove ha_statistic_increment calls
They have been replaced with increment_statistics() calls made by handler::ha_XXX() methods.
1 parent 3af9986 commit 2770e80

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

storage/rocksdb/ha_rocksdb.cc

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6246,7 +6246,6 @@ int ha_rocksdb::index_read_map_impl(uchar *buf, const uchar *key,
62466246
int rc= 0;
62476247
DBUG_ENTER("ha_rocksdb::index_read_map");
62486248

6249-
ha_statistic_increment(&SSV::ha_read_key_count);
62506249
const std::shared_ptr<const Rdb_key_def>& kd= m_key_descr_arr[active_index];
62516250
uint actual_key_parts= kd->get_key_parts();
62526251
bool using_full_key= is_using_full_key(keypart_map, actual_key_parts);
@@ -6791,7 +6790,6 @@ int ha_rocksdb::get_row_by_rowid(uchar *buf, const char *rowid,
67916790
int ha_rocksdb::index_next(uchar *buf)
67926791
{
67936792
bool moves_forward= true;
6794-
ha_statistic_increment(&SSV::ha_read_next_count);
67956793
if (m_key_descr_arr[active_index]->m_is_reverse_cf)
67966794
{
67976795
moves_forward= false;
@@ -6807,7 +6805,6 @@ int ha_rocksdb::index_next(uchar *buf)
68076805
int ha_rocksdb::index_prev(uchar *buf)
68086806
{
68096807
bool moves_forward= false;
6810-
ha_statistic_increment(&SSV::ha_read_prev_count);
68116808
if (m_key_descr_arr[active_index]->m_is_reverse_cf)
68126809
{
68136810
moves_forward= true;
@@ -6855,7 +6852,6 @@ int ha_rocksdb::index_next_with_direction(uchar *buf, bool move_forward)
68556852
int ha_rocksdb::index_first(uchar *buf)
68566853
{
68576854
m_sk_match_prefix= nullptr;
6858-
ha_statistic_increment(&SSV::ha_read_first_count);
68596855
int rc= m_key_descr_arr[active_index]->m_is_reverse_cf ?
68606856
index_last_intern(buf) : index_first_intern(buf);
68616857
if (rc == HA_ERR_KEY_NOT_FOUND)
@@ -6867,7 +6863,6 @@ int ha_rocksdb::index_first(uchar *buf)
68676863
int ha_rocksdb::index_last(uchar *buf)
68686864
{
68696865
m_sk_match_prefix= nullptr;
6870-
ha_statistic_increment(&SSV::ha_read_last_count);
68716866
int rc= m_key_descr_arr[active_index]->m_is_reverse_cf ?
68726867
index_first_intern(buf) : index_last_intern(buf);
68736868
if (rc == HA_ERR_KEY_NOT_FOUND)
@@ -7253,7 +7248,6 @@ int ha_rocksdb::write_row(uchar *buf)
72537248
DBUG_ENTER("ha_rocksdb::write_row");
72547249
DBUG_ASSERT(buf == table->record[0]);
72557250
DBUG_ASSERT(m_lock_rows == RDB_LOCK_WRITE);
7256-
ha_statistic_increment(&SSV::ha_write_count);
72577251
/*
72587252
Note: "buf == table->record[0]" is copied from innodb. I am not aware of
72597253
any use cases where this condition is not true.
@@ -8022,7 +8016,6 @@ int ha_rocksdb::rnd_init(bool scan)
80228016
int ha_rocksdb::rnd_next(uchar *buf)
80238017
{
80248018
int rc;
8025-
ha_statistic_increment(&SSV::ha_read_rnd_next_count);
80268019
for (;;)
80278020
{
80288021
rc= rnd_next_with_direction(buf, true);
@@ -8205,7 +8198,6 @@ int ha_rocksdb::delete_row(const uchar *buf)
82058198
{
82068199
DBUG_ENTER("ha_rocksdb::delete_row");
82078200

8208-
ha_statistic_increment(&SSV::ha_delete_count);
82098201
set_last_rowkey(buf);
82108202

82118203
rocksdb::Slice key_slice(m_last_rowkey.ptr(), m_last_rowkey.length());
@@ -8436,7 +8428,6 @@ int ha_rocksdb::rnd_pos(uchar *buf, uchar *pos)
84368428
size_t len;
84378429
DBUG_ENTER("ha_rocksdb::rnd_pos");
84388430

8439-
ha_statistic_increment(&SSV::ha_read_rnd_count);
84408431
stats.rows_requested++;
84418432
len= m_pk_descr->key_length(table, rocksdb::Slice((const char*)pos,
84428433
ref_length));
@@ -8507,7 +8498,6 @@ int ha_rocksdb::update_row(const uchar *old_data, uchar *new_data)
85078498
*/
85088499
DBUG_ASSERT(new_data == table->record[0]);
85098500

8510-
ha_statistic_increment(&SSV::ha_update_count);
85118501
int rv;
85128502
rv= update_write_row(old_data, new_data, false);
85138503

0 commit comments

Comments
 (0)