Skip to content

Commit f559baa

Browse files
author
wuxianrong
committed
Resolve core issues at runtime
1 parent 5c2ecdb commit f559baa

2 files changed

Lines changed: 9 additions & 4 deletions

File tree

include/pika_cmd_table_manager.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ class PikaCmdTableManager {
7373
*/
7474
std::unordered_map<std::string, CommandStatistics> cmdstat_map_;
7575
std::unordered_map<std::string, CommandStatistics> slow_command_count_;
76-
std::mutex command_mutex_;
7776
std::shared_mutex histograms_mutex_;
7877
std::shared_mutex slow_command_mutex_;
7978
std::shared_ptr<prometheus::Registry> prometheus_registry_;

src/pika_cmd_table_manager.cc

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,15 @@
1515
extern std::unique_ptr<PikaConf> g_pika_conf;
1616

1717
void PikaCmdTableManager::ResetCommandCount() {
18-
std::lock_guard<std::mutex> lock(command_mutex_);
19-
slow_command_count_.clear();
20-
InitHistograms();
18+
{
19+
std::unique_lock<std::shared_mutex> write_lock(slow_command_mutex_);
20+
slow_command_count_.clear();
21+
}
22+
23+
{
24+
std::unique_lock<std::shared_mutex> write_lock(histograms_mutex_);
25+
InitHistograms();
26+
}
2127
}
2228

2329
void PikaCmdTableManager::InitHistograms() {

0 commit comments

Comments
 (0)