MB_21787: Remove log message from couch-kvstore for non-existent key 45/70345/2
authorDave Rigby <daver@couchbase.com>
Thu, 24 Nov 2016 14:54:55 +0000 (14:54 +0000)
committerDave Rigby <daver@couchbase.com>
Fri, 25 Nov 2016 07:56:15 +0000 (07:56 +0000)
If an arithmetic operation is performed on a key which isn't resident,
on a full-eviction bucket then the following message is often printed
in the logs:

    WARNING (default) Failed to fetch data from database, vBucket=0
    key=Key_19 error=document not found [none]

This is unnecessarily verbose - this is not actually a warning in this
case and can quickly fill up the logs.

(Note: the reason this is shown is that prior to full-eviction, all
metadata would be resident and hence it would be unexpected to attempt
to read from disk a key which doesn't exist). However this is no
longer the case).

Change-Id: I6ac4f2b1f8d43700ff6869da8a6670e21454b3a0
Reviewed-on: http://review.couchbase.org/70345
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Jim Walker <jim@couchbase.com>
src/couch-kvstore/couch-kvstore.cc

index c8b5551..6a7273a 100644 (file)
@@ -2096,10 +2096,6 @@ int CouchKVStore::getMultiCb(Db *db, DocInfo *docinfo, void *ctx) {
     couchstore_error_t errCode = cbCtx->cks.fetchDoc(db, docinfo, returnVal,
                                                      cbCtx->vbId, meta_only);
     if (errCode != COUCHSTORE_SUCCESS && !meta_only) {
-        LOG(EXTENSION_LOG_WARNING, "Failed to fetch data from database, "
-            "vBucket=%d key=%s error=%s [%s]", cbCtx->vbId,
-            keyStr.c_str(), couchstore_strerror(errCode),
-            couchkvstore_strerrno(db, errCode).c_str());
         st.numGetFailure++;
     }