atomic_cache 0.5.3.rc1 → 0.5.4.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 508ff6dd988d3b864438c2c54d3bca4cba2490a62c2ae97ed2b313522d5e8694
-  data.tar.gz: e0cd9a214c089780758955d2642c406484e0404c669d87faf347de1d8cb3a781
+  metadata.gz: e912c124c9879b3208c849a21f6f82a665ddc0f3ab1e268a1355d6cfb7bda54a
+  data.tar.gz: 3fc8b8814852f95414e8c699fb2b6f17d4c0a0516f7bc12dbdb2d167096604ac
 SHA512:
-  metadata.gz: f5e751211a50a029cf4043fa89bf74e50cb200835985c884b5a0faa782b9fc3b8b4c9a0b55be561763363c6625e124c9c11bee75cb15596ddea7b7213bb20bc7
-  data.tar.gz: b3fa0f5bed38fe417103a02409fcb1bf9ccb7611d69887ac6b5cc565e1440a6148ab9f306cb3283fbce2369bf26012f2532a21e889871a47e22d3a517f8b9b84
+  metadata.gz: 5104f60b2f4f9d2816a53e7a19dcf58eda5e245b052c1d048c586cae8df985e0eb17f58c13bdc8dc07db4bb50607d033cb57c7a701db50a896955575cc710e3b
+  data.tar.gz: bae2d03aac48378a6075d648c9131cf265271192658e30b5191a6bb82735e7a4aae0162d62b0c4153ed46ad16d9375cf8c832bd899b408e3ab7a9617710d2f71

data/docs/MODEL_SETUP.md

@@ -29,3 +29,65 @@ class Foo < ActiveRecord::Base
   cache_version(5)
 end
 ```
+
+### Inheritance
+
+When either `force_cache_class` or `cache_version` are used, those values will always be preferred by classes which inherit from the class on which those macros exist. When macros are used on descendant classes, the "closet" value wins.
+
+#### Example #1
+The keys used by `Bar` will be prefixed with `custom:v5`, as it prefers both the 'custom' class name and version 5 from the parent.
+```ruby
+class Foo < ActiveRecord::Base
+  include AtomicCache::GlobalLMTCacheConcern
+  force_cache_class('custom')
+  cache_version(5)
+end
+
+class Bar < Foo
+end
+```
+
+#### Example #2
+The keys used by `Bar` will be prefixed with `bar:v5`, as the version 5 is taken from the parent, but the use of forcing on the child class result in the cache class of 'bar'.
+```ruby
+class Foo < ActiveRecord::Base
+  include AtomicCache::GlobalLMTCacheConcern
+  cache_version(5)
+end
+
+class Bar < Foo
+  force_cache_class('bar')
+end
+```
+
+#### Example #3
+The keys used by `Bar` will be still be prefixed with `bar:v5` for the same reasons as above.
+```ruby
+class Foo < ActiveRecord::Base
+  include AtomicCache::GlobalLMTCacheConcern
+  force_cache_class('custom')
+  cache_version(5)
+end
+
+class Bar < Foo
+  force_cache_class('bar')
+end
+```
+
+### Rails Model Inheritance
+
+It's not uncommon in rails to end up with model inhertiance. For example:
+
+```ruby
+class Content < ActiveRecord::Base
+  include AtomicCache::GlobalLMTCacheConcern
+  force_cache_class('content')
+end
+
+class BlogPost < Content
+end
+```
+
+If these models will be cached together into a single key, it's preferable to force the cache class on the parent, causing all the descendant types to use the same keyspace. Not doing this will cause each subtype to use it's own last modified time.
+
+If the models will be treated as separate collections and cached separately, this is not recommended. Alternately, if only some subtypes will be cached together, those should share a forced cache class and version.

data/lib/atomic_cache/concerns/global_lmt_cache_concern.rb

@@ -63,15 +63,29 @@ module AtomicCache
         @timestamp_manager.last_modified_time
       end
 
+      # protected
+
+      def cache_version_get
+        return @atomic_cache_version if @atomic_cache_version.present?
+        return self.superclass.cache_version_get if self.superclass.respond_to?(:cache_version_get)
+        nil
+      end
+
+      def cache_class_get
+        return @atomic_cache_class if @atomic_cache_class.present?
+        return self.superclass.cache_class_get if self.superclass.respond_to?(:cache_class_get)
+        nil
+      end
+
       private
 
       def init_atomic_cache
         return if @atomic_cache.present?
         ATOMIC_CACHE_CONCERN_MUTEX.synchronize do
-          cache_class = @atomic_cache_class || default_cache_class
+          cache_class = cache_class_get || default_cache_class
           prefix = [cache_class]
           prefix.unshift(DefaultConfig.instance.namespace) if DefaultConfig.instance.namespace.present?
-          prefix.push("v#{@atomic_cache_version}") if @atomic_cache_version.present?
+          prefix.push("v#{cache_version_get}") if cache_version_get.present?
           @default_cache_keyspace = Keyspace.new(namespace: prefix, root: cache_class)
 
           @timestamp_manager = LastModTimeKeyManager.new(

data/lib/atomic_cache/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AtomicCache
-  VERSION = "0.5.3.rc1"
+  VERSION = "0.5.4.rc1"
 end

data/spec/atomic_cache/concerns/global_lmt_cache_concern_spec.rb

@@ -113,6 +113,61 @@ describe 'AtomicCacheConcern' do
       subject.expire_cache
       expect(key_storage.store).to have_key(:'foo:v3:lmt')
     end
+
+    context 'with version < class inheritance' do
+      subject do
+        class Foo3
+          include AtomicCache::GlobalLMTCacheConcern
+          cache_version(6)
+        end
+        class Foo4 < Foo3
+          force_cache_class('foo')
+        end
+        Foo4
+      end
+
+      it 'uses the version from the parent and the forced class name from the macro' do
+        subject.expire_cache
+        expect(key_storage.store).to have_key(:'foo:v6:lmt')
+      end
+    end
+
+    context 'with class < version inheritance' do
+      subject do
+        class Foo5
+          include AtomicCache::GlobalLMTCacheConcern
+          force_cache_class('foo')
+        end
+        class Foo6 < Foo5
+          cache_version(6)
+        end
+        Foo6
+      end
+
+      it 'uses the cache class from the parent and version from the macro' do
+        subject.expire_cache
+        expect(key_storage.store).to have_key(:'foo:v6:lmt')
+      end
+    end
+
+    context 'dual force_cache_class' do
+      subject do
+        class Foo6
+          include AtomicCache::GlobalLMTCacheConcern
+          force_cache_class('foo')
+        end
+        class Foo7 < Foo6
+          force_cache_class('bar')
+          cache_version(6)
+        end
+        Foo7
+      end
+
+      it 'uses the prefers the forced class name from the macro' do
+        subject.expire_cache
+        expect(key_storage.store).to have_key(:'bar:v6:lmt')
+      end
+    end
   end
 
   context 'storage macros' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: atomic_cache
 version: !ruby/object:Gem::Version
-  version: 0.5.3.rc1
+  version: 0.5.4.rc1
 platform: ruby
 authors:
 - Ibotta Developers
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-08-09 00:00:00.000000000 Z
+date: 2021-08-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -137,6 +137,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.1
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement