common/TrackedOp: check tracking_enabled for event "initiated/done".

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
(cherry picked from commit d7585c020de41f25e6a598661844169f2bdbb2b4)

diff --git a/src/common/TrackedOp.cc b/src/common/TrackedOp.cc
index 1875648fe488ab789dc5bf5e60960f7efe380808..10c0a960ee247c62bebcf0f9df40d52ec1b5be87 100644 (file)
--- a/src/common/TrackedOp.cc
+++ b/src/common/TrackedOp.cc
@@ -290,12 +290,12 @@ void OpTracker::_mark_event(TrackedOp *op, const string &evt,
 }
 
 void OpTracker::RemoveOnDelete::operator()(TrackedOp *op) {
-  op->mark_event("done");
   if (!tracker->tracking_enabled) {
     op->_unregistered();
     delete op;
     return;
   }
+  op->mark_event("done");
   tracker->unregister_inflight_op(op);
   // Do not delete op, unregister_inflight_op took control
 }

diff --git a/src/common/TrackedOp.h b/src/common/TrackedOp.h
index 42d1eafe7005d67881629503c2196b292dd1b1fe..b2015b88f9604fac0c0215d16bf6d1a9ba30a44b 100644 (file)
--- a/src/common/TrackedOp.h
+++ b/src/common/TrackedOp.h
@@ -161,7 +161,8 @@ protected:
     warn_interval_multiplier(1)
   {
     tracker->register_inflight_op(&xitem);
-    events.push_back(make_pair(initiated_at, "initiated"));
+    if (tracker->tracking_enabled)
+      events.push_back(make_pair(initiated_at, "initiated"));
   }
 
   /// output any type-specific data you want to get when dump() is called