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 9e83c0b74b6c8bf29d1629f8253dc311f71935c5..ddb2f91ac3f55980633c84a8615c842ba47b544f 100644 (file)
--- 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);