crimson/osd: use fmt::format() for formatting temp recovery oid

for more concise code and better readability

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/crimson/osd/recovery_backend.cc b/src/crimson/osd/recovery_backend.cc
index 9a07342f5db7d50e901e0ba9403ce1241412c400..4bd43429a48bda3a2134de8a5a2bfb1c64e5644f 100644 (file)
--- a/src/crimson/osd/recovery_backend.cc
+++ b/src/crimson/osd/recovery_backend.cc
@@ -19,12 +19,14 @@ namespace {
 
 hobject_t RecoveryBackend::get_temp_recovery_object(
   const hobject_t& target,
-  eversion_t version)
+  eversion_t version) const
 {
-  ostringstream ss;
-  ss << "temp_recovering_" << pg.get_info().pgid << "_" << version
-    << "_" << pg.get_info().history.same_interval_since << "_" << target.snap;
-  hobject_t hoid = target.make_temp_hobject(ss.str());
+  hobject_t hoid =
+    target.make_temp_hobject(fmt::format("temp_recovering_{}_{}_{}_{}",
+                                         pg.get_info().pgid,
+                                         version,
+                                         pg.get_info().history.same_interval_since,
+                                         target.snap));
   logger().debug("{} {}", __func__, hoid);
   return hoid;
 }

diff --git a/src/crimson/osd/recovery_backend.h b/src/crimson/osd/recovery_backend.h
index 2bffaed6a6413e0e377ebe887cca95d35470b28e..cb0ae9f20565f15828c6284fb4209d20d6a7f644 100644 (file)
--- a/src/crimson/osd/recovery_backend.h
+++ b/src/crimson/osd/recovery_backend.h
@@ -188,7 +188,7 @@ protected:
   std::map<hobject_t, WaitForObjectRecovery> recovering;
   hobject_t get_temp_recovery_object(
     const hobject_t& target,
-    eversion_t version);
+    eversion_t version) const;
 
   boost::container::flat_set<hobject_t> temp_contents;