Switch ThreadWheelTimekeeperHighRes to HeapTimekeeper

Summary: `ThreadWheelTimekeeperHighRes` is being deprecated, since `HeapTimekeeper` is both more efficient and more accurate (see analysis in D49060468). `HeapTimekeeper` is already the default timekeeper for futures. Reviewed By: stuclar Differential Revision: D49318889 fbshipit-source-id: 9c44492df5d310f27e4a5f4b782c82c5e806b3eb
facebook · Sep 20, 2023 · c25ff7f · c25ff7f
1 parent 5a04fb3
commit c25ff7f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/mcrouter/routes/LatencyInjectionRoute.cpp b/mcrouter/routes/LatencyInjectionRoute.cpp
@@ -8,13 +8,17 @@
 #include "mcrouter/routes/LatencyInjectionRoute.h"
 
 #include <folly/Singleton.h>
+#include <folly/futures/HeapTimekeeper.h>
 
 namespace facebook::memcache::mcrouter {
 
 namespace {
-folly::Singleton<folly::ThreadWheelTimekeeperHighRes>
+
+struct McrouterTimekeeperTag {};
+folly::Singleton<folly::HeapTimekeeper, McrouterTimekeeperTag>
     timekeeperHighResSingleton_;
-}
+
+} // namespace
 
 namespace detail {
 folly::ReadMostlySharedPtr<folly::Timekeeper> getTimekeeperHighResSingleton() {

diff --git a/mcrouter/routes/LatencyInjectionRoute.h b/mcrouter/routes/LatencyInjectionRoute.h
@@ -14,7 +14,6 @@
 #include <folly/fibers/Baton.h>
 
 #include <folly/experimental/ReadMostlySharedPtr.h>
-#include <folly/experimental/ThreadWheelTimekeeperHighRes.h>
 
 #include "mcrouter/config.h"
 #include "mcrouter/lib/RouteHandleTraverser.h"