EP buckets in full eviction mode can end up with incorrect high item count

void VBucket::deletedOnDiskCbk(const Item& queuedItem, bool deleted) {

    auto res = ht.findItem(queuedItem);

    auto* v = res.storedValue;

    // Delete the item in the hash table iff:

    //  1. Item is existent in hashtable, and deleted flag is true

    //  2. rev seqno of queued item matches rev seqno of hash table item

    if (v && v->isDeleted() && (queuedItem.getRevSeqno() == v->getRevSeqno())) {

        EP_LOG_ERR("VBucket::deletedOnDiskCbk deleteStoredValue() key:{}",

                   queuedItem.getKey().c_str());

        bool isDeleted = deleteStoredValue(res.lock, *v);

        if (!isDeleted) {

            throw std::logic_error(

                    "deletedOnDiskCbk:callback: "

                    "Failed to delete key with seqno:" +

                    std::to_string(v->getBySeqno()) + "' from bucket " +

                    std::to_string(res.lock.getBucketNum()));

        }

        /**

         * Deleted items are to be added to the bloomfilter,

         * in either eviction policy.

         */

        addToFilter(queuedItem.getKey());

    }

    if (deleted) {

        ++stats.totalPersisted;

        ++opsDelete;

        /**

         * MB-30137: Decrement the total number of on-disk items. This needs to

         * be done to ensure that the item count is accurate in the case of full

         * eviction. We should only decrement the counter for committed (via

         * mutation or commit) items as we only increment for these.

         */

        if (v && queuedItem.isCommitted()) {

            decrNumTotalItems();

        }

    }

...

}