magick-feature-flags 0.9.9 → 0.9.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 97522d2ffcbf4267b4c5fbb6b8243eef1a91d2e00eee70717d6a5cfd74c6b3f4
-  data.tar.gz: 68a31983ffc8b7926c740a6444e4f1d189e7e7eff55a0fbbd175a9c3bee08e0a
+  metadata.gz: 6ff57666debab09c055a1709e27902b4ff03780b62ca97382e3f0df191b4b1ce
+  data.tar.gz: 1420d03c05427c4c0ef7f319debb108542b36fa5fccf999ad13e3bd0571c0451
 SHA512:
-  metadata.gz: ccf360255814ca54a8d7ab84f13329d74fbdbb5c61b9943053b607fe16107999bcbddf3954c8b4b6b9ec08c0ffcf5aea547132e18969eb1295d67a0e97ce3baf
-  data.tar.gz: e31e42e107f303988590f20bd3f3098221ae024a2f2eedefc5ebcf04781e76f845e10d672ab8e086eef91a3d675b6f2c6f21adfa7edb64e2fd7c413ab4b008e3
+  metadata.gz: 2150805b902ed92fc1f033493e47697541078d21fb7d9d1e827ddf8b22d9e17fd58e8e1ed926aa47fecf1662c491d910ecba528a0d1ce8f306b340c9e0b35a16
+  data.tar.gz: 21144c7b0ad4c6c23eb379f088898757f438058dc9c2278f3fc30a03e188b03a6b58e9cb4aac8d5e37127523b5bcfc562a00c08079de95da035d10b9afa28ad5

data/lib/magick/adapters/registry.rb

@@ -97,6 +97,13 @@ module Magick
         !redis_adapter.nil?
       end
 
+      # Get Redis client (public method for use by other classes)
+      def redis_client
+        return nil unless redis_adapter
+
+        redis_adapter.instance_variable_get(:@redis)
+      end
+
       private
 
       attr_reader :memory_adapter, :redis_adapter, :circuit_breaker

data/lib/magick/performance_metrics.rb

@@ -65,9 +65,7 @@ module Magick
       should_flush = false
       @mutex.synchronize do
         # Flush if we have enough pending updates or enough time has passed
-        if @pending_updates.size >= @batch_size || (Time.now - @last_flush) >= @flush_interval
-          should_flush = true
-        end
+        should_flush = true if @pending_updates.size >= @batch_size || (Time.now - @last_flush) >= @flush_interval
       end
 
       flush_to_redis if should_flush
@@ -88,13 +86,13 @@ module Magick
       # Update Redis in batch
       begin
         adapter = Magick.adapter_registry || Magick.default_adapter_registry
-        if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_adapter
-          redis = adapter.redis_adapter.instance_variable_get(:@redis)
+        if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_available?
+          redis = adapter.redis_client
           if redis
             updates_to_flush.each do |feature_name, count|
               redis_key = "magick:stats:#{feature_name}"
               redis.incrby(redis_key, count)
-              redis.expire(redis_key, 86400 * 7) # Keep stats for 7 days
+              redis.expire(redis_key, 86_400 * 7) # Keep stats for 7 days
             end
           end
         end
@@ -130,8 +128,8 @@ module Magick
       if @redis_enabled
         begin
           adapter = Magick.adapter_registry || Magick.default_adapter_registry
-          if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_adapter
-            redis = adapter.redis_adapter.instance_variable_get(:@redis)
+          if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_available?
+            redis = adapter.redis_client
             if redis
               redis_key = "magick:stats:#{feature_name_str}"
               redis_count = redis.get(redis_key).to_i
@@ -152,11 +150,11 @@ module Magick
       if @redis_enabled
         begin
           adapter = Magick.adapter_registry || Magick.default_adapter_registry
-          if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_adapter
-            redis = adapter.redis_adapter.instance_variable_get(:@redis)
+          if adapter.is_a?(Magick::Adapters::Registry) && adapter.redis_available?
+            redis = adapter.redis_client
             if redis
               # Get all stats keys
-              redis.keys("magick:stats:*").each do |key|
+              redis.keys('magick:stats:*').each do |key|
                 feature_name = key.to_s.sub('magick:stats:', '')
                 redis_count = redis.get(key).to_i
                 combined_counts[feature_name] = (combined_counts[feature_name] || 0) + redis_count

data/lib/magick/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Magick
-  VERSION = '0.9.9'
+  VERSION = '0.9.10'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: magick-feature-flags
 version: !ruby/object:Gem::Version
-  version: 0.9.9
+  version: 0.9.10
 platform: ruby
 authors:
 - Andrew Lobanov