diff options
| author | Tor Brede Vekterli <vekterli@yahooinc.com> | 2021-11-25 12:26:46 +0000 |
|---|---|---|
| committer | Tor Brede Vekterli <vekterli@yahooinc.com> | 2021-11-25 12:26:46 +0000 |
| commit | 144a49b76b8f85294baf04520537e95605deb8da (patch) | |
| tree | 4ba8a001ed6230b10c8ecf3eb2580056b6b6d5ad /storage | |
| parent | 67ea18c1dca8e096f5a3c00f075c6835fc80624f (diff) | |
Reduce log level from error to warning for update inconsistency message
`error` level is for the "node is on fire and falling down an elevator
shaft" type of messages, not for operation inconsistencies. Reduce
to `warning` accordingly.
Diffstat (limited to 'storage')
| -rw-r--r-- | storage/src/vespa/storage/distributor/operations/external/updateoperation.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/storage/src/vespa/storage/distributor/operations/external/updateoperation.cpp b/storage/src/vespa/storage/distributor/operations/external/updateoperation.cpp index 3aa231e1790..a16eef0ab6f 100644 --- a/storage/src/vespa/storage/distributor/operations/external/updateoperation.cpp +++ b/storage/src/vespa/storage/distributor/operations/external/updateoperation.cpp @@ -155,13 +155,13 @@ UpdateOperation::onReceive(DistributorStripeMessageSender& sender, for (uint32_t i = 0; i < _results.size(); i++) { if (_results[i].oldTs < oldTs) { - LOG(error, "Update operation for '%s' in bucket %s updated documents with different timestamps. " - "This should not happen and may indicate undetected replica divergence. " - "Found ts=%" PRIu64 " on node %u, ts=%" PRIu64 " on node %u", - reply.getDocumentId().toString().c_str(), - reply.getBucket().toString().c_str(), - _results[i].oldTs, _results[i].nodeId, - _results[goodNode].oldTs, _results[goodNode].nodeId); + LOG(warning, "Update operation for '%s' in bucket %s updated documents with different timestamps. " + "This should not happen and may indicate undetected replica divergence. " + "Found ts=%" PRIu64 " on node %u, ts=%" PRIu64 " on node %u", + reply.getDocumentId().toString().c_str(), + reply.getBucket().toString().c_str(), + _results[i].oldTs, _results[i].nodeId, + _results[goodNode].oldTs, _results[goodNode].nodeId); _metrics.diverging_timestamp_updates.inc(); replyToSend.setNodeWithNewestTimestamp(_results[goodNode].nodeId); |