Add class comments and resolve PR comments.

1 files changed, 4 insertions, 6 deletions

diff --git a/storage/src/vespa/storage/persistence/persistenceutil.cpp b/storage/src/vespa/storage/persistence/persistenceutil.cpp
index c46b9ab7059..9095b351f64 100644
--- a/storage/src/vespa/storage/persistence/persistenceutil.cpp
+++ b/storage/src/vespa/storage/persistence/persistenceutil.cpp
@@ -52,12 +52,7 @@ MessageTracker::setMetric(FileStorThreadMetrics::Op& metric) {
     _metric = &metric;
 }
 
-MessageTracker::~MessageTracker()
-{
-    if (_reply.get() && _reply->getResult().success()) {
-        _metric->latency.addValue(_timer.getElapsedTimeAsDouble());
-    }
-}
+MessageTracker::~MessageTracker() = default;
 
 void
 MessageTracker::sendReply() {
@@ -70,6 +65,9 @@ MessageTracker::sendReply() {
                 _env.setBucketInfo(*this, _bucketLock->getBucket());
             }
         }
+        if (getReply().getResult().success()) {
+            _metric->latency.addValue(_timer.getElapsedTimeAsDouble());
+        }
         LOG(spam, "Sending reply up: %s %" PRIu64,
             getReply().toString().c_str(), getReply().getMsgId());
         _env._fileStorHandler.sendReply(std::move(_reply));