kredis 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8f1ad5de0aa1b64c083109609ec185ea81e35b3550fced4347f5bff12ff35832
-  data.tar.gz: 85d287eb15524b0b668524a51820ffb3a64d4bccda398f07802abb5867b49b92
+  metadata.gz: f3e3619ec3a977b472348572202b324c813d550d43cc866ec1aab7dad8e463f0
+  data.tar.gz: 920c49f0cfe443a71dd5dc8f112070ab8061bbe2d352c98b45026ceeed871f2e
 SHA512:
-  metadata.gz: cfb7cfa924d9a6ecb79febf81cc2b2479694b0e275219d53e031b01fe2296a23862eee96f6359d05f33fc333808638b39c187c9f641252de3435488df3f6e372
-  data.tar.gz: cd06423690991d2a5364804e240aac27b88f792bc951623ce1e5f4f5a17946b817a05cfda9a4defb831e8e6c1d79ee0c3ab8523342d94c3a2259caca8609f1d9
+  metadata.gz: 8f97bda6b251a100411c0338d2760aa1c2565f3ee1901f5f4971123cbe8b7a49fabf64ccc210775e9694263fd1a2a142641bcf81659776f746d05163640afda5
+  data.tar.gz: 546b9246e3952a92452fe55cf574233b12d50f29bdeb345c7efe3ebf86bd3ab9a6ff4d22731438ae3258586caca530c1e7229ffee9b581a4a22b04f585dd4467

data/README.md

@@ -2,7 +2,7 @@
 
 Kredis (Keyed Redis) encapsulates higher-level types and data structures around a single key, so you can interact with them as coherent objects rather than isolated procedural commands. These higher-level structures can be configured as attributes within Active Models and Active Records using a declarative DSL.
 
-Kredis is configured using env-aware yaml files, using `Rails.application.config_for`, so you can locate the data structures on separate redis instances, if you've reached a scale where a single shared instance is no longer sufficient.
+Kredis is configured using env-aware YAML files, using `Rails.application.config_for`, so you can locate the data structures on separate Redis instances, if you've reached a scale where a single shared instance is no longer sufficient.
 
 Kredis provides namespacing support for keys such that you can safely run parallel testing against the data structures without different tests trampling each others data.
 
@@ -154,7 +154,7 @@ test:
   <<: *development
 ```
 
-Additional configurations can be added under `config/redis/*.yml` and referenced when a type is created.
+Additional configurations can be added under `config/redis/*.yml` and referenced when a type is created, e.g. `Kredis.string("mystring", config: :strings)` would lookup `config/redis/strings.yml`. Under the hood `Kredis.configured_for` is called which'll pass the configuration on to `Redis.new`.
 
 
 ## License

data/lib/kredis/migration.rb

@@ -41,7 +41,7 @@ class Kredis::Migration
       cursor = "0"
       begin
         cursor, keys = @redis.scan(cursor, match: key_pattern, count: SCAN_BATCH_SIZE)
-        @redis.multi { yield keys }
+        @redis.pipelined { yield keys }
       end until cursor == "0"
     end
 

data/lib/kredis/railtie.rb

@@ -6,8 +6,8 @@ module Kredis
 
     initializer "kredis.testing" do
       ActiveSupport.on_load(:active_support_test_case) do
-        parallelize_setup    { |worker| Kredis.namespace = "test-#{worker}" }
-        parallelize_teardown { Kredis.clear_all }
+        parallelize_setup { |worker| Kredis.namespace = "test-#{worker}" }
+        teardown { Kredis.clear_all }
       end
     end
 

data/lib/kredis/type_casting.rb

@@ -16,7 +16,7 @@ module Kredis::TypeCasting
     when TrueClass, FalseClass
       value ? "t" : "f"
     when Time, DateTime, ActiveSupport::TimeWithZone
-      value.to_f
+      value.iso8601(9)
     when Hash
       JSON.dump(value)
     else
@@ -33,7 +33,7 @@ module Kredis::TypeCasting
     when :decimal     then value.to_d
     when :float       then value.to_f
     when :boolean     then value == "t" ? true : false
-    when :datetime    then Time.at(value.to_i)
+    when :datetime    then Time.iso8601(value)
     when :json        then JSON.load(value)
     end if value.present?
   end

data/lib/kredis/types/proxy.rb

@@ -1,6 +1,7 @@
-require "active_support/core_ext/enumerable"
-
 class Kredis::Types::Proxy
+  require_relative "proxy/failsafe"
+  include Failsafe
+
   attr_accessor :redis, :key
 
   def initialize(redis, key, **options)
@@ -13,14 +14,16 @@ class Kredis::Types::Proxy
   end
 
   def method_missing(method, *args, **kwargs)
-    Kredis.logger&.debug log_message(method, *args, **kwargs)
-    redis.public_send method, key, *args, **kwargs
+    failsafe do
+      Kredis.logger&.debug log_message(method, *args, **kwargs)
+      redis.public_send method, key, *args, **kwargs
+    end
   end
 
   private
     def log_message(method, *args, **kwargs)
-      args      = args.flatten.compact_blank.presence
-      kwargs    = kwargs.compact_blank.presence
+      args      = args.flatten.reject(&:blank?).presence
+      kwargs    = kwargs.reject { |_k, v| v.blank? }.presence
       type_name = self.class.name.split("::").last
 
       "[Kredis #{type_name}] #{method.upcase} #{key} #{args&.inspect} #{kwargs&.inspect}".chomp

data/lib/kredis/types/proxy/failsafe.rb

@@ -0,0 +1,26 @@
+module Kredis::Types::Proxy::Failsafe
+  def initialize(*)
+    super
+    @fail_safe_suppressed = false
+  end
+
+  def failsafe
+    yield
+  rescue Redis::BaseError
+    raise if fail_safe_suppressed?
+  end
+
+  def suppress_failsafe_with(returning: nil)
+    old_fail_safe_suppressed, @fail_safe_suppressed = @fail_safe_suppressed, true
+    yield
+  rescue Redis::BaseError
+    returning
+  ensure
+    @fail_safe_suppressed = old_fail_safe_suppressed
+  end
+
+  private
+    def fail_safe_suppressed?
+      @fail_safe_suppressed
+    end
+end

data/lib/kredis/types/proxying.rb

@@ -13,6 +13,10 @@ class Kredis::Types::Proxying
     options.each { |key, value| send("#{key}=", value) }
   end
 
+  def failsafe(returning: nil, &block)
+    proxy.suppress_failsafe_with(returning: returning, &block)
+  end
+
   private
     delegate :type_to_string, :string_to_type, :types_to_strings, :strings_to_types, to: :Kredis
 end

data/lib/kredis/types/set.rb

@@ -29,7 +29,7 @@ class Kredis::Types::Set < Kredis::Types::Proxying
   end
 
   def size
-    scard
+    scard.to_i
   end
 
   def take

data/lib/kredis/types/slots.rb

@@ -6,23 +6,25 @@ class Kredis::Types::Slots < Kredis::Types::Proxying
   attr_accessor :available
 
   def reserve
-    if block_given?
-      begin
-        if reserve
-          yield
+    failsafe returning: false do
+      if block_given?
+        begin
+          if reserve
+            yield
+            true
+          else
+            false
+          end
+        ensure
+          release
+        end
+      else
+        if incr <= available
           true
         else
+          release
           false
         end
-      ensure
-        release
-      end
-    else
-      if incr <= available
-        true
-      else
-        release
-        false
       end
     end
   end
@@ -32,7 +34,9 @@ class Kredis::Types::Slots < Kredis::Types::Proxying
   end
 
   def available?
-    get.to_i < available
+    failsafe returning: false do
+      get.to_i < available
+    end
   end
 
   def reset

data/lib/kredis/version.rb

@@ -1,3 +1,3 @@
 module Kredis
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: kredis
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Kasper Timm Hansen
 - David Heinemeier Hansson
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-03 00:00:00.000000000 Z
+date: 2021-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -39,7 +39,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '4.2'
-description: 
+description:
 email: david@hey.com
 executables: []
 extensions: []
@@ -60,6 +60,7 @@ files:
 - lib/kredis/types/flag.rb
 - lib/kredis/types/list.rb
 - lib/kredis/types/proxy.rb
+- lib/kredis/types/proxy/failsafe.rb
 - lib/kredis/types/proxying.rb
 - lib/kredis/types/scalar.rb
 - lib/kredis/types/set.rb
@@ -70,7 +71,7 @@ homepage: https://github.com/rails/kredis
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -86,7 +87,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Higher-level data structures built on Redis.
 test_files: []