incrdecr_cached_counts 0.0.6 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: a018b5815304845fe6bd35505944da125418b3e5
-  data.tar.gz: c1799c3ddaa64683faf943ea9e0d352177c896f0
+SHA256:
+  metadata.gz: b1abd3ec58583d46e713effab128bece6d90b4799261cb5714981e8e545a293b
+  data.tar.gz: 435c1c883ba00f766473e563732602245e5a1680f92bbc852251ce48497bab53
 SHA512:
-  metadata.gz: d4ae781d8e7394523beb4f91cbb622f018bc9ee30a04a92cbd4e00e7bdecbeb87f93c9f87dacc6b27a41872c1b1b64f97349a4ecfb1a89043345922a8f57b699
-  data.tar.gz: d52e2154bc2bd201b8297b9ca32cd4b142f1a265e2f7f89b8df3b86aa4c0b8ccd79ecf76e847da13c0f86389ca017ef465ae501d330a22653fb13521e73e1862
+  metadata.gz: 870a0f140a0d815f1e2e824ae5c9cd802cfcab3be446f376ad2f08f80ff2eadd560d899a7f85cfd099d08a2ffbc517beeb5926575cae557e5a68c2d3d9bb79e8
+  data.tar.gz: 6f6058c68108b32c94e3d6c4be7d785974371f21b7fd828f4df9a5b33d108912537ed102685bb2601aac5e3267a32ea27d4a65508721c0ca5cb5566e44aac1f5

data/.circleci/config.yml

@@ -0,0 +1,47 @@
+version: 2.1
+
+jobs:
+  build:
+    docker:
+      - image: cimg/ruby:2.7-node # use a tailored CircleCI docker image.
+    steps:
+      - checkout
+
+      - run: bundle config set path 'vendor/bundle'
+      - run: sudo apt-get update
+      - run: sudo apt install libsqlite3-dev
+
+      # There's probably a better way of doing this that avoids the repetition and manual
+      # deletion of Gemfile.lock.
+      - restore_cache:
+          key: rails-5.2.4.4-{{ checksum "Gemfile" }}
+      - run:
+          name: bundle install
+          command: bundle install
+          environment:
+            ACTIVERECORD_VERSION: 5.2.4.4
+      - save_cache:
+          key: rails-5.2.4.4-{{ checksum "Gemfile" }}
+          paths:
+            - vendor/bundle
+      - run:
+          name: Test with Rails 5.2
+          command: bundle exec rspec
+
+      - run: rm Gemfile.lock
+      - restore_cache:
+          key: rails-6.0.3.4-{{ checksum "Gemfile" }}
+      - run:
+          name: bundle install
+          command: bundle install --path vendor/bundle
+          environment:
+            ACTIVERECORD_VERSION: 6.0.3.4
+      - save_cache:
+          key: rails-6.0.3.4-{{ checksum "Gemfile" }}
+          paths:
+            - vendor/bundle
+      - restore_cache:
+          key: rails-6.0.3.4-{{ checksum "Gemfile" }}
+      - run:
+          name: Test with Rails 6.0
+          command: bundle exec rspec

data/README.rdoc

@@ -1,5 +1,7 @@
 = CachedCounts
 
+{<img src="http://circleci.com/gh/academia-edu/cached_counts.svg?style=svg" />}[https://circleci.com/gh/academia-edu/cached_counts]
+
 A replacement for Rails' counter caches, using memcached.
 
 Caches counts of models in a scope or association in memcached, and keeps the

data/incrdecr_cached_counts.gemspec

@@ -19,13 +19,11 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_dependency "rails", ">= 4.0"
+  s.add_dependency "rails", ">= 5.2"
   s.add_dependency "dalli"
 
   s.add_development_dependency "sqlite3"
   s.add_development_dependency "rspec"
   s.add_development_dependency "database_cleaner"
-  s.add_development_dependency "test_after_commit"
-  s.add_development_dependency "after_commit_exception_notification"
   s.add_development_dependency "rake"
 end

data/lib/cached_counts.rb

@@ -1,6 +1,6 @@
 require 'cached_counts/logger'
 require 'cached_counts/dalli_check'
-require 'cached_counts/connection_for'
+require 'cached_counts/query_context'
 
 module CachedCounts
   extend ActiveSupport::Concern
@@ -185,6 +185,18 @@ module CachedCounts
           association_count_key(id, attribute_name, version)
         end
 
+        # Try to fetch values for ids from the cache. If it's a miss return the default value
+        define_singleton_method "try_#{attr_name}_counts_for" do |ids, default=nil|
+          raw_result = Rails.cache.read_multi(*ids.map{|id| association_count_key(id, attribute_name, version)})
+
+          result = {}
+          ids.each do |id|
+            result[id] = raw_result[association_count_key(id, attribute_name, version)]&.to_i || default
+          end
+
+          result
+        end
+
         define_singleton_method "#{attr_name}_count_for" do |id|
           new({id: id}, without_protection: true).send("#{attr_name}_count")
         end
@@ -255,11 +267,8 @@ module CachedCounts
           relation = relation.reorder('')
           relation.select_values = ['count(*)']
 
-          conn = CachedCounts.connection_for(counted_class)
-          if Rails.version < '4.2'.freeze
-            conn.select_value(relation.to_sql, nil, relation.values[:bind] || []).to_i
-          else
-            conn.select_value(relation.to_sql).to_i
+          CachedCounts.query_context.call(counted_class) do
+            counted_class.connection.select_value(relation.to_sql).to_i
           end
         end
 
@@ -285,7 +294,7 @@ module CachedCounts
     end
 
     def add_counting_hooks(attribute_name, key_getter, counted_class, options)
-      increment_hook = "increment_#{attribute_name}_count"
+      increment_hook = "increment_#{attribute_name}_count".to_sym
       counted_class.send :define_method, increment_hook do
         if (key = instance_exec &key_getter)
           Rails.cache.increment(
@@ -296,7 +305,7 @@ module CachedCounts
         end
       end
 
-      decrement_hook = "decrement_#{attribute_name}_count"
+      decrement_hook = "decrement_#{attribute_name}_count".to_sym
       counted_class.send :define_method, decrement_hook do
         if (key = instance_exec &key_getter)
           Rails.cache.decrement(

data/lib/cached_counts/query_context.rb

@@ -0,0 +1,21 @@
+module CachedCounts
+  class << self
+    # Optional configuration: Set a proc which takes as arguments (1) the class
+    # we are counting and (2) the block that runs the count query. The
+    # `query_context` block must call the block that's passed as an argument.
+    #
+    # This is useful for replication, e.g.,
+    #
+    #     CachedCounts.query_context = proc do |klass, &run_query|
+    #       role = klass == User ? :reading : :writing
+    #       ActiveRecord::Base.connected_to(role: role) do
+    #         run_query.call
+    #       end
+    #     end
+    attr_writer :query_context
+
+    def query_context
+      @query_context ||= proc { |_klass, &block| block.call }
+    end
+  end
+end

data/lib/cached_counts/version.rb

@@ -1,3 +1,3 @@
 module CachedCounts
-  VERSION = "0.0.6"
+  VERSION = "0.4.0"
 end

data/spec/spec_helper.rb

@@ -25,13 +25,6 @@ end
 require_relative './fixtures.rb'
 require 'database_cleaner'
 
-if Rails.version.to_f < 5.0
-  require 'test_after_commit'
-end
-if Rails.version.to_f < 4.2
-  require 'after_commit_exception_notification'
-end
-
 RSpec.configure do |config|
   config.before(:suite) do
     DatabaseCleaner.strategy = :transaction

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: incrdecr_cached_counts
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.4.0
 platform: ruby
 authors:
 - David Judd
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-23 00:00:00.000000000 Z
+date: 2020-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: dalli
   requirement: !ruby/object:Gem::Requirement
@@ -80,34 +80,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: test_after_commit
-  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: after_commit_exception_notification
-  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: rake
   requirement: !ruby/object:Gem::Requirement
@@ -130,20 +102,18 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".circleci-matrix.yml"
+- ".circleci/config.yml"
 - ".gitignore"
-- ".travis.yml"
 - ".yardopts"
 - Gemfile
 - MIT-LICENSE
 - README.rdoc
 - Rakefile
-- circle.yml
 - incrdecr_cached_counts.gemspec
 - lib/cached_counts.rb
-- lib/cached_counts/connection_for.rb
 - lib/cached_counts/dalli_check.rb
 - lib/cached_counts/logger.rb
+- lib/cached_counts/query_context.rb
 - lib/cached_counts/railtie.rb
 - lib/cached_counts/version.rb
 - spec/caches_count_of_spec.rb
@@ -174,19 +144,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: A replacement for Rails' counter caches using memcached (via Dalli)
-test_files:
-- spec/caches_count_of_spec.rb
-- spec/caches_count_where_spec.rb
-- spec/database.yml
-- spec/fixtures.rb
-- spec/fixtures/department.rb
-- spec/fixtures/following.rb
-- spec/fixtures/university.rb
-- spec/fixtures/user.rb
-- spec/spec_helper.rb
-has_rdoc: 
+test_files: []

data/.circleci-matrix.yml

@@ -1,8 +0,0 @@
-env:
-  - ACTIVERECORD_VERSION=4.2.6
-  - ACTIVERECORD_VERSION=4.1.15
-  - ACTIVERECORD_VERSION=4.0.13
-
-command:
-  - rm -f Gemfile.lock && ACTIVERECORD_VERSION=$ACTIVERECORD_VERSION bundle
-  - bundle exec rake

data/.travis.yml

@@ -1,15 +0,0 @@
-sudo: false # use newer travis infrastructure
-language: ruby
-rvm:
-  - "2.1.8"
-  - "2.2.4"
-  - "2.3.0"
-env:
-  - "ACTIVERECORD_VERSION=4.2.6"
-  - "ACTIVERECORD_VERSION=4.1.15"
-  - "ACTIVERECORD_VERSION=4.0.13"
-  - "ACTIVERECORD_VERSION=master"
-matrix:
-  allow_failures:
-    # Master may or may not pass
-    - env: "ACTIVERECORD_VERSION=master"

data/circle.yml

@@ -1,10 +0,0 @@
-machine:
-  ruby:
-    version: 2.2.3
-
-dependencies:
-  pre:
-    - curl -fsSL https://git.io/v2Ifn | bash
-
-database:
-  override:

data/lib/cached_counts/connection_for.rb

@@ -1,14 +0,0 @@
-module CachedCounts
-  class << self
-    # Optional configuration: set a proc which takes the class we are counting
-    # and returns a connection. Useful with replication, e.g. with Octopus:
-    #
-    # `CachedCounts.connection_for = proc { |klass| klass.using(:read_slave).connection }`
-    attr_writer :connection_for
-
-    def connection_for(counted_class)
-      @connection_for ||= proc { |klass| klass.connection }
-      @connection_for[counted_class]
-    end
-  end
-end