Introduce option to enable cache lookup monitoring

Author: Daniel Hartnell

lib/graphql/fragment_cache.rb

@@ -25,6 +25,7 @@ module FragmentCache
     class << self
       attr_reader :cache_store
       attr_accessor :enabled
+      attr_accessor :monitoring_enabled
       attr_accessor :namespace
       attr_accessor :default_options
 
@@ -87,6 +88,7 @@ def verify_interpreter_and_analysis!(schema_defn)
 
     self.cache_store = MemoryStore.new
     self.enabled = true
+    self.monitoring_enabled = false
     self.namespace = "graphql"
     self.default_options = {}
     self.skip_cache_when_query_has_errors = false

lib/graphql/fragment_cache/fragment.rb

@@ -30,17 +30,19 @@ def read_multi(fragments)
             FragmentCache.cache_store.read_multi(*cache_keys)
           end
 
-          begin
-            fragments.map do |fragment|
-              cache_lookup_event(
-                cache_key: fragment.cache_key,
-                operation_name: fragment.context.query.operation_name,
-                path: fragment.path,
-                cache_hit: cache_keys_to_values.key?(fragment.cache_key)
-              )
+          if GraphQL::FragmentCache.monitoring_enabled
+            begin
+              fragments.map do |fragment|
+                cache_lookup_event(
+                  cache_key: fragment.cache_key,
+                  operation_name: fragment.context.query.operation_name,
+                  path: fragment.path,
+                  cache_hit: cache_keys_to_values.key?(fragment.cache_key)
+                )
+              end
+            rescue
+              # Allow cache_lookup_event to fail when we do not have all of the requested attributes
             end
-          rescue
-            # Allow cache_lookup_event to fail when we do not have all of the requested attributes
           end
 
           # Fragmenst without values or with renew_cache: true in their context will have nil values like the read method