Skip to content

Commit f8840f9

Browse files
maykovGunnar Kudrjavets
authored andcommitted
Fixing the build break with PSI on
Summary: MixedOther Jenkins configuration builds with PSI turned on. This exposed the problems which this diff fixes. MySQLOnRocksDB/mysql-5.6#28 Test Plan: built Reviewers: jonahcohen, hermanlee4 Differential Revision: https://reviews.facebook.net/D33603
1 parent 08abd0a commit f8840f9

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

storage/rocksdb/ha_rocksdb.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -672,8 +672,8 @@ static void init_rocksdb_psi_keys()
672672
count= array_elements(all_rocksdb_stages);
673673
mysql_stage_register(category, all_rocksdb_stages, count);
674674

675-
count= array_elements(all_server_threads);
676-
mysql_thread_register(category, all_server_threads, count);
675+
count= array_elements(all_rocksdb_threads);
676+
mysql_thread_register(category, all_rocksdb_threads, count);
677677
}
678678
#endif
679679

storage/rocksdb/rdb_cf_manager.cc

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,14 @@ bool is_cf_name_reverse(const char *name)
4141
return false;
4242
}
4343

44+
#ifdef HAVE_PSI_INTERFACE
45+
static PSI_mutex_key ex_key_cfm;
46+
#endif
4447

4548
void Column_family_manager::init(std::vector<std::string> *names,
4649
std::vector<rocksdb::ColumnFamilyHandle*> *handles)
4750
{
48-
mysql_mutex_init(NULL, &cfm_mutex, MY_MUTEX_INIT_FAST);
51+
mysql_mutex_init(ex_key_cfm, &cfm_mutex, MY_MUTEX_INIT_FAST);
4952
DBUG_ASSERT(names->size() == handles->size());
5053
DBUG_ASSERT(names->size() > 0);
5154

0 commit comments

Comments
 (0)