]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
crimson/osd/ops_executor: queue pg log entry for CLONE at the front
authorXuehan Xu <xuxuehan@qianxin.com>
Tue, 29 Aug 2023 06:05:02 +0000 (14:05 +0800)
committerXuehan Xu <xuxuehan@qianxin.com>
Tue, 7 Nov 2023 09:10:57 +0000 (17:10 +0800)
This is consistent with the order of ops in ceph::os::Transaction

Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>
src/crimson/osd/ops_executer.cc

index 868a7a3fb13d25c1b7ff3c2cd82768b9984df900..09b1a492576fe12d196d6eb12c5c7076fa943789 100644 (file)
@@ -970,7 +970,7 @@ void OpsExecuter::CloningContext::apply_to(
   ObjectContext& processed_obc) &&
 {
   log_entry.mtime = processed_obc.obs.oi.mtime;
-  log_entries.emplace_back(std::move(log_entry));
+  log_entries.insert(log_entries.begin(), std::move(log_entry));
   processed_obc.ssc->snapset = std::move(new_snapset);
 }
 
@@ -985,7 +985,7 @@ OpsExecuter::flush_clone_metadata(
   auto maybe_snap_mapped = interruptor::now();
   if (cloning_ctx) {
     std::move(*cloning_ctx).apply_to(log_entries, *obc);
-    const auto& coid = log_entries.back().soid;
+    const auto& coid = log_entries.front().soid;
     const auto& cloned_snaps = obc->ssc->snapset.clone_snaps[coid.snap];
     maybe_snap_mapped = snap_map_clone(
       coid,