semian 0.25.4 → 0.26.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f4179609ac096cef76c5e8492229c7c374d1dbc26ae8c758ae79e18ab9595813
-  data.tar.gz: f6fe881485adcb7142e33d1bcb4036f3adc782c357f5c865aa2d67a97878118b
+  metadata.gz: 20f2ab860d074c5420db9d4a5ec675307fdf046c4ad06d3aab227273b7a3716c
+  data.tar.gz: 25e865fa1a72948765a572a20ccfbdea783c4940fd245bf18f281af8a2351f54
 SHA512:
-  metadata.gz: 5254f05b90dc20305c6cc0ee855d3a0593fe13f95648a523c281cdd9ac9056e138a124947137b028647dbc8b9b0c049fd387e8c5562032a5a5be622793f6741d
-  data.tar.gz: 5de64c3000d0f5d59d68df7ff4d67ecaee233c6528bb1498648f960bcd59dc6b43e543baf12a6e7927eabc0392bb44c2c1dbeb1d55861ef1a2515cb3f09cc32e
+  metadata.gz: f8dbe833518cadd4083cb104554add1f1e0b506d02f92b8b55dbc763ff822e5e5fdf22633b3de415fde54ba56780c5ea2e2aae88bd08d7367d5fa5e10ea58d33
+  data.tar.gz: a126b289713f04a264a7a8e24e25fa1b7cd964477a22ec5f62696710c79abb9f6e73c74ef71ed77e7d05665d79ea67b53bc598d0c8b2a837eef8f8a75b608e71

data/lib/semian/instrumentable.rb

@@ -1,14 +1,22 @@
 # frozen_string_literal: true
 
+require "concurrent-ruby"
+
 module Semian
   module Instrumentable
+    SUBSCRIBERS = Concurrent::Map.new
+
     def subscribe(name = rand, &block)
-      subscribers[name] = block
+      SUBSCRIBERS[name] = block
       name
     end
 
     def unsubscribe(name)
-      subscribers.delete(name)
+      SUBSCRIBERS.delete(name)
+    end
+
+    def subscribers
+      SUBSCRIBERS
     end
 
     # Args:
@@ -18,13 +26,7 @@ module Semian
     #   adapter (string)
     #   payload (optional)
     def notify(*args)
-      subscribers.values.each { |subscriber| subscriber.call(*args) }
-    end
-
-    private
-
-    def subscribers
-      @subscribers ||= {}
+      SUBSCRIBERS.values.each { |subscriber| subscriber.call(*args) }
     end
   end
 end

data/lib/semian/simple_integer.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "thread"
+require "concurrent"
 
 module Semian
   module Simple
@@ -26,14 +27,29 @@ module Semian
   end
 
   module ThreadSafe
-    class Integer < Simple::Integer
-      def initialize(*)
-        super
-        @lock = Mutex.new
+    class Integer
+      def initialize
+        @atom = Concurrent::AtomicFixnum.new(0)
+      end
+
+      def value
+        @atom.value
       end
 
-      def increment(*)
-        @lock.synchronize { super }
+      def value=(new_value)
+        @atom.value = new_value
+      end
+
+      def increment(val = 1)
+        @atom.increment(val)
+      end
+
+      def reset
+        @atom.value = 0
+      end
+
+      def destroy
+        reset
       end
     end
   end

data/lib/semian/simple_sliding_window.rb

@@ -1,69 +1,62 @@
 # frozen_string_literal: true
 
 require "thread"
+require "concurrent"
 
 module Semian
-  module Simple
-    class SlidingWindow # :nodoc:
-      extend Forwardable
+  module SlidingWindowBehavior
+    extend Forwardable
 
-      def_delegators :@window, :size, :last, :empty?
-      attr_reader :max_size
+    def_delegators :@window, :size, :last, :empty?
+    attr_reader :max_size
 
-      # A sliding window is a structure that stores the most @max_size recent timestamps
-      # like this: if @max_size = 4, current time is 10, @window =[5,7,9,10].
-      # Another push of (11) at 11 sec would make @window [7,9,10,11], shifting off 5.
+    # A sliding window is a structure that stores the most @max_size recent timestamps
+    # like this: if @max_size = 4, current time is 10, @window =[5,7,9,10].
+    # Another push of (11) at 11 sec would make @window [7,9,10,11], shifting off 5.
 
-      def initialize(max_size:)
-        @max_size = max_size
-        @window = []
-      end
+    def reject!(&block)
+      @window.reject!(&block)
+      self
+    end
 
-      def reject!(&block)
-        @window.reject!(&block)
-      end
+    def push(value)
+      resize_to(@max_size - 1) # make room
+      @window << value
+      self
+    end
+    alias_method :<<, :push
 
-      def push(value)
-        resize_to(@max_size - 1) # make room
-        @window << value
-        self
-      end
-      alias_method :<<, :push
+    def clear
+      @window.clear
+      self
+    end
+    alias_method :destroy, :clear
 
-      def clear
-        @window.clear
-        self
-      end
-      alias_method :destroy, :clear
+    private
+
+    def resize_to(size)
+      @window = @window.last(size) if @window.size >= size
+    end
+  end
 
-      private
+  module Simple
+    class SlidingWindow # :nodoc:
+      include SlidingWindowBehavior
 
-      def resize_to(size)
-        @window = @window.last(size) if @window.size >= size
+      def initialize(max_size:)
+        @max_size = max_size
+        @window = []
       end
     end
   end
 
   module ThreadSafe
-    class SlidingWindow < Simple::SlidingWindow
-      def initialize(**)
-        super
-        @lock = Mutex.new
-      end
-
-      # #size, #last, and #clear are not wrapped in a mutex. For the first two,
-      # the worst-case is a thread-switch at a timing where they'd receive an
-      # out-of-date value--which could happen with a mutex as well.
-      #
-      # As for clear, it's an all or nothing operation. Doesn't matter if we
-      # have the lock or not.
-
-      def reject!(*)
-        @lock.synchronize { super }
-      end
+    class SlidingWindow
+      include SlidingWindowBehavior
 
-      def push(*)
-        @lock.synchronize { super }
+      def initialize(max_size:)
+        @max_size = max_size
+        @window = Concurrent::Array.new
       end
     end
   end

data/lib/semian/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Semian
-  VERSION = "0.25.4"
+  VERSION = "0.26.5"
 end

data/lib/semian.rb

@@ -4,6 +4,7 @@ require "forwardable"
 require "logger"
 require "weakref"
 require "thread"
+require "concurrent-ruby"
 
 require "semian/version"
 require "semian/instrumentable"
@@ -110,6 +111,20 @@ module Semian
   self.default_permissions = 0660
   self.default_force_config_validation = false
 
+  # We only allow disabling thread-safety for parts of the code that are on the hot path.
+  # Since locking there could have a significant impact. Everything else is enforced thread safety
+  def thread_safe?
+    return @thread_safe if defined?(@thread_safe)
+
+    @thread_safe = true
+  end
+
+  def thread_safe=(thread_safe)
+    @thread_safe = thread_safe
+  end
+
+  @reset_mutex = Mutex.new
+
   def issue_disabled_semaphores_warning
     return if defined?(@warning_issued)
 
@@ -200,7 +215,7 @@ module Semian
     # of who the consumer was so that we can clear the resource reference if needed.
     consumer = args.delete(:consumer)
     if consumer&.class&.include?(Semian::Adapter) && !args[:dynamic]
-      consumer_set = (consumers[name] ||= ObjectSpace::WeakMap.new)
+      consumer_set = consumers.compute_if_absent(name) { ObjectSpace::WeakMap.new }
       consumer_set[consumer] = true
     end
     self[name] || register(name, **args)
@@ -233,7 +248,7 @@ module Semian
     resource = resources.delete(name)
     if resource
       resource.bulkhead&.unregister_worker
-      consumers_for_resource = consumers.delete(name) || {}
+      consumers_for_resource = consumers.delete(name) || ObjectSpace::WeakMap.new
       consumers_for_resource.each_key(&:clear_semian_resource)
     end
   end
@@ -245,29 +260,11 @@ module Semian
     end
   end
 
-  # Retrieves a hash of all registered resources.
-  def resources
-    @resources ||= LRUHash.new
-  end
-
-  # Retrieves a hash of all registered resource consumers.
-  def consumers
-    @consumers ||= {}
-  end
-
   def reset!
-    @consumers = {}
-    @resources = LRUHash.new
-  end
-
-  def thread_safe?
-    return @thread_safe if defined?(@thread_safe)
-
-    @thread_safe = true
-  end
-
-  def thread_safe=(thread_safe)
-    @thread_safe = thread_safe
+    @reset_mutex.synchronize do
+      @consumers = Concurrent::Map.new
+      @resources = LRUHash.new
+    end
   end
 
   THREAD_BULKHEAD_DISABLED_VAR = :semian_bulkheads_disabled
@@ -285,6 +282,22 @@ module Semian
     thread.thread_variable_set(THREAD_BULKHEAD_DISABLED_VAR, old_value)
   end
 
+  def resources
+    return @resources if defined?(@resources) && @resources
+
+    @reset_mutex.synchronize do
+      @resources ||= LRUHash.new
+    end
+  end
+
+  def consumers
+    return @consumers if defined?(@consumers) && @consumers
+
+    @reset_mutex.synchronize do
+      @consumers ||= Concurrent::Map.new
+    end
+  end
+
   private
 
   def create_circuit_breaker(name, **options)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: semian
 version: !ruby/object:Gem::Version
-  version: 0.25.4
+  version: 0.26.5
 platform: ruby
 authors:
 - Scott Francis