From: Samuel Just <sam.just@inktank.com>
Date: Sat, 1 Mar 2014 21:55:24 +0000 (-0800)
Subject: TrackedOp: clear_payload as well in unregister_inflight_op
X-Git-Tag: v0.78~106^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=718cda6e95d85a6a159ca951b0257f0054689345;p=ceph.git

TrackedOp: clear_payload as well in unregister_inflight_op

We want to minimize the cost of maintaining the historic ops.

Signed-off-by: Samuel Just <sam.just@inktank.com>
---

diff --git a/src/common/TrackedOp.cc b/src/common/TrackedOp.cc
index 9e83c0b74b6c..ddb2f91ac3f5 100644
--- a/src/common/TrackedOp.cc
+++ b/src/common/TrackedOp.cc
@@ -120,8 +120,9 @@ void OpTracker::unregister_inflight_op(TrackedOp *i)
 {
   // caller checks;
   assert(tracking_enabled);
-  i->request->clear_data();
 
+  i->request->clear_data();
+  i->request->clear_payload();
 
   Mutex::Locker locker(ops_in_flight_lock);
   assert(i->xitem.get_list() == &ops_in_flight);