ReplicatedPG: don't copy the ops vector into OpContext

There's no need to copy it, and if we refer to the same vector from
the original MOSDOp, we can modify rvals for individual ops, since the
MOSDOp ops vector is copied to the MOSDOpReply when it is constructed.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

diff --git a/src/osd/ReplicatedPG.h b/src/osd/ReplicatedPG.h
index b6a80e2594c8676bc0af6e25ef1114c2168a359a..8e0876e58bb527945e1744db75e4156c377f789f 100644 (file)
--- a/src/osd/ReplicatedPG.h
+++ b/src/osd/ReplicatedPG.h
@@ -409,7 +409,7 @@ public:
   struct OpContext {
     OpRequestRef op;
     osd_reqid_t reqid;
-    vector<OSDOp> ops;
+    vector<OSDOp> &ops;
 
     const ObjectState *obs; // Old objectstate
     const SnapSet *snapset; // Old snapset