diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2020-05-04 21:21:27 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2020-05-04 22:24:50 +0000 |
commit | 2078bee591af22f091e7e9a9d7c039b0fca2be5c (patch) | |
tree | 6022e0e2f7e42ef23162b7bcf6e51b88a8df4640 | |
parent | 9d769dca411691680c16e73094ef27958b260959 (diff) |
Only generate reply if it is a command.
-rw-r--r-- | storage/src/vespa/storage/persistence/persistenceutil.cpp | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/storage/src/vespa/storage/persistence/persistenceutil.cpp b/storage/src/vespa/storage/persistence/persistenceutil.cpp index c8dd564e390..72d6a1bb8d3 100644 --- a/storage/src/vespa/storage/persistence/persistenceutil.cpp +++ b/storage/src/vespa/storage/persistence/persistenceutil.cpp @@ -29,7 +29,7 @@ namespace { (id == api::MessageType::REMOVELOCATION_ID || id == api::MessageType::JOINBUCKETS_ID)); } - constexpr double WARN_ON_SLOW_OPERATIONS = 5.0; + const vespalib::duration WARN_ON_SLOW_OPERATIONS = 5s; } MessageTracker::MessageTracker(PersistenceUtil & env, @@ -70,18 +70,20 @@ MessageTracker::~MessageTracker() = default; void MessageTracker::sendReply() { - generateReply(static_cast<api::StorageCommand &>(*_msg)); + if ( ! _msg->getType().isReply()) { + generateReply(static_cast<api::StorageCommand &>(*_msg)); + } if ((hasReply() && getReply().getResult().failed()) || getResult().failed()) { _env._metrics.failedOperations.inc(); } - double duration = _timer.getElapsedTimeAsDouble(); + vespalib::duration duration = vespalib::from_s(_timer.getElapsedTimeAsDouble()/1000.0); if (duration >= WARN_ON_SLOW_OPERATIONS) { LOGBT(warning, _msg->getType().toString(), - "Slow processing of message %s on disk %u. Processing time: %4.0f ms (>=%4.0f ms)", - _msg->toString().c_str(), _env._partition, duration, WARN_ON_SLOW_OPERATIONS); + "Slow processing of message %s on disk %u. Processing time: %1.1f s (>=%1.1f s)", + _msg->toString().c_str(), _env._partition, vespalib::to_s(duration), vespalib::to_s(WARN_ON_SLOW_OPERATIONS)); } else { - LOGBT(spam, _msg->getType().toString(), "Processing time of message %s on disk %u: %4.0f ms", - _msg->toString(true).c_str(), _env._partition, duration); + LOGBT(spam, _msg->getType().toString(), "Processing time of message %s on disk %u: %1.1f s", + _msg->toString(true).c_str(), _env._partition, vespalib::to_s(duration)); } if (hasReply()) { if ( ! _context.getTrace().getRoot().isEmpty()) { |