SERVER-44978 Move the "cache generation" to AuthorizationManagerImpl

The "cache generation" concept is something specific to the
Authorization Manager and doesn't fit with an asynchronous (non
totally-ordered cache). This change moves it out of the ReadThroughCache
and makes it an AuthorizationManager concept.

1 files changed, 2 insertions, 10 deletions

diff --git a/src/mongo/util/read_through_cache.h b/src/mongo/util/read_through_cache.h
index 54e5d4bfc45..bf8b377f6ec 100644
--- a/src/mongo/util/read_through_cache.h
+++ b/src/mongo/util/read_through_cache.h
@@ -48,12 +48,6 @@ class ReadThroughCacheBase {
     ReadThroughCacheBase(const ReadThroughCacheBase&) = delete;
     ReadThroughCacheBase& operator=(const ReadThroughCacheBase&) = delete;
 
-public:
-    /**
-     * Returns the cache generation identifier.
-     */
-    OID getCacheGeneration() const;
-
 protected:
     ReadThroughCacheBase(Mutex& mutex, ServiceContext* service, ThreadPoolInterface& threadPool);
 
@@ -380,13 +374,11 @@ public:
     void invalidateIf(const Pred& predicate) {
         CacheGuard guard(this);
         _updateCacheGeneration(guard);
-        _cache.invalidateIf([&](const Key& key, const StoredValue* storedValue) {
-            return predicate(key, &storedValue->value);
-        });
+        _cache.invalidateIf([&](const Key& key, const StoredValue*) { return predicate(key); });
     }
 
     void invalidateAll() {
-        invalidateIf([](const Key&, const Value*) { return true; });
+        invalidateIf([](const Key&) { return true; });
     }
 
     /**