Move FCC::onWorkerThread into .cpp

Summary: We don't need this in the header, move it to the source instead.  Reduces the visible interface and possibly helps w/ compilation times.

Reviewed By: AlnisM

Differential Revision: D91154246

fbshipit-source-id: ee177f92905eea4cd629e40e21cafbdae3f6c054

Author: rlyerly

cachelib/interface/components/FlashCacheComponent.cpp

@@ -17,6 +17,7 @@
 #include "cachelib/interface/components/FlashCacheComponent.h"
 
 #include "cachelib/common/Time.h"
+#include "cachelib/interface/utils/CoroFiberAdapter.h"
 
 using namespace facebook::cachelib::navy;
 
@@ -130,6 +131,17 @@ class FlashCacheItem : public CacheItem {
   }
 }
 
+template <typename FuncT, typename ReturnT, typename CleanupFuncT>
+folly::coro::Task<ReturnT> FlashCacheComponent::onWorkerThread(
+    FuncT&& func, CleanupFuncT&& cleanup) {
+  XDCHECK(!cache_->regionManager_.isOnWorker())
+      << "Calling public APIs from a worker thread is unsupported";
+  co_return co_await utils::onWorkerThread(
+      cache_->regionManager_.getNextWorker(),
+      std::forward<FuncT>(func),
+      std::forward<CleanupFuncT>(cleanup));
+}
+
 const std::string& FlashCacheComponent::getName() const noexcept {
   return name_;
 }

cachelib/interface/components/FlashCacheComponent.h

@@ -17,7 +17,6 @@
 #pragma once
 
 #include "cachelib/interface/CacheComponent.h"
-#include "cachelib/interface/utils/CoroFiberAdapter.h"
 #include "cachelib/navy/block_cache/BlockCache.h"
 
 namespace facebook::cachelib::interface {
@@ -74,14 +73,7 @@ class FlashCacheComponent : public CacheComponent {
             typename ReturnT = std::invoke_result_t<FuncT>,
             typename CleanupFuncT = std::function<void(ReturnT)>>
   folly::coro::Task<ReturnT> onWorkerThread(FuncT&& func,
-                                            CleanupFuncT&& cleanup = {}) {
-    XDCHECK(!cache_->regionManager_.isOnWorker())
-        << "Calling public APIs from a worker thread is unsupported";
-    co_return co_await utils::onWorkerThread(
-        cache_->regionManager_.getNextWorker(),
-        std::forward<FuncT>(func),
-        std::forward<CleanupFuncT>(cleanup));
-  }
+                                            CleanupFuncT&& cleanup = {});
 
   using AllocData =
       std::tuple<navy::RegionDescriptor, uint32_t, navy::RelAddress>;