circuitbox 2.0.0.pre3 → 2.0.0.pre4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 91bcf88d24630e489540fd2c149ce6b468ecdd7e
-  data.tar.gz: 8576e094971fb74bb390b2a1508fd6c330644915
+SHA256:
+  metadata.gz: fb89d6ef46d73dfa35e7c43181112309aa524979ec1285688cc8cebf8d781bd4
+  data.tar.gz: 735c23115ea5127923c05b84d5b6cfe64193521bcd751639cc6ceab32c6b97ce
 SHA512:
-  metadata.gz: 7e2246d5b01dddaa5fb1e3637f5f66a1bb1a9ffcbf20d8bec5d28d787328f748b7dc1818a72662ede92aab94eae32f1608eacc42697cd0fbe72f5ef16f7c42c9
-  data.tar.gz: 9b2022385d22e3e6fc31a1be8028fa047a49fb59f09779c77bdae53712ab8cda482513f6dd7909a1ee3b04ca4a6fdcb1d9a110cc57febb1b5e712b19f284b4a0
+  metadata.gz: a29d10725e8a1a36eec12985681cd66b7c2b3303cb996bfe72226674230d7c8ef38b02bb5731665edd49c7922b2a9f0072434a2ab33da7f64713875415071420
+  data.tar.gz: 9a165096e2b0e5fe484c433284a3c0b6658f4b880552602cb57b22fcf3ea540302b3d3a0910b31b8f3e1938259ca6394faefd3bffffc6c40bf3104d8d02564ec

data/README.md

@@ -1,6 +1,6 @@
 # Circuitbox
 
-[![Build Status](https://travis-ci.org/yammer/circuitbox.svg?branch=master)](https://travis-ci.org/yammer/circuitbox) [![Gem Version](https://badge.fury.io/rb/circuitbox.svg)](https://badge.fury.io/rb/circuitbox)
+![Tests](https://github.com/yammer/circuitbox/workflows/Tests/badge.svg) [![Gem Version](https://badge.fury.io/rb/circuitbox.svg)](https://badge.fury.io/rb/circuitbox)
 
 Circuitbox is a Ruby circuit breaker gem. It protects your application from failures of its service dependencies. It wraps calls to external services and monitors for failures in one minute intervals. Once more than 10 requests have been made with a 50% failure rate, Circuitbox stops sending requests to that failing service for one minute. This helps your application gracefully degrade.
 Resources about the circuit breaker pattern:
@@ -43,7 +43,7 @@ Using the `run` method will throw an exception when the circuit is open or the u
 ```
 
 ## Global Configuration
-Circuitbox has defaults for circuit_store, notifier, timer, and logger.
+Circuitbox has defaults for circuit_store, notifier, and logger.
 This can be configured through ```Circuitbox.configure```.
 The circuit cache used by ```Circuitbox.circuit``` will be cleared after running ```Circuitbox.configure```.
 This means when accessing the circuit through ```Circuitbox.circuit``` any custom configuration options should always be given.
@@ -55,7 +55,6 @@ will need to be recreated to pick up the new defaults.
   Circuitbox.configure do |config|
     config.default_circuit_store = Circuitbox::MemoryStore.new
     config.default_notifier = Circuitbox::Notifier::Null.new
-    config.default_timer = Circuitbox::Timer::Simple.new
     config.default_logger = Rails.logger
   end
 ```
@@ -91,12 +90,6 @@ class ExampleServiceClient
       # This overrides what is set in the global configuration
       logger: Logger.new(STDOUT),
 
-      # Customized Timer object
-      # Use NullTimer if you don't want to time circuit execution
-      # Use MonotonicTimer to avoid bad time metrics on system time resync
-      # This overrides what is set in the global configuration
-      execution_timer: SimpleTimer,
-
       # Customized notifier
       # overrides the default
       # this overrides what is set in the global configuration
@@ -168,7 +161,7 @@ end
 
 `payload[:gauge]` can be:
 
-- `execution_time` # execution time will only be notified when circuit is closed and block is successfully executed.
+- `runtime` # runtime will only be notified when circuit is closed and block is successfully executed.
 
 **warnings:**
 in case of misconfiguration, circuitbox will fire a circuitbox_warning
@@ -241,71 +234,6 @@ c.use Circuitbox::FaradayMiddleware, circuit_breaker_options: {}
 c.use Circuitbox::FaradayMiddleware, open_circuit: lambda { |response| response.status >= 500 }
 ```
 
-## CHANGELOG
-### v1.1.0
-- ruby 2.2 support [#58](https://github.com/yammer/circuitbox/pull/58)
-- configurable logger [#58](https://github.com/yammer/circuitbox/pull/58)
-
-### v1.0.3
-- fix timeout issue for default configuration, as default `:Memory` adapter does
-  not natively support expires, we need to actually load it on demand.
-- fix memoization of `circuit_breaker_options` not actually doing memoization in
-  `excon` and `faraday` middleware.
-
-### v1.0.2
-- Fix timeout issue [#51](https://github.com/yammer/circuitbox/issues/51)
-  [sebastian-juliu](https://github.com/sebastian-julius)
-
-### v1.0.1
-- Fix Rails integration, as version 1.0.0 removed the rails tasks integration, but missed
-  removing the related railtie.
-
-### v1.0.0
-- support for cross process circuitbreakers by swapping the circuitbreaker store for a
-  `Moneta` supported key value store.
-- Change `FaradayMiddleware` default behaviour to not open on `4xx` errors but just on `5xx`
-  server errors and connection errors
-- Remove stat store, which was largely unused
-
-### v0.11.0
-- fix URI require missing (https://github.com/yammer/circuitbox/pull/42 @gottfrois)
-- configurable circuitbox store backend via Moneta supporting multi process circuits
-
-### v0.10.4
-- Issue #39, keep the original backtrace for the wrapped exception around when
-  re-raising a Circuitbox::Error
-
-### v0.10.3
-- Circuitbox::ServiceFailureError wraps the original exception that was raised.
-  The behaviour for to_s wasn't exposing this information and was returning the
-  name of class "Circuitbox::ServiceFailureError". Change the behaviour for to_s
-  to indicate this exception is a wrapper around the original exception.
-  [sherrry](https://github.com/sherrry)
-
-### v0.10.2
-- Faraday middleware passes two arguments to the `default_value` callback, not
-  just one.  First argument is still the error response from Faraday if there is
-  one.  Second argument is the exception that caused the call to fail if it
-  failed before Faraday returned a response.  Old behaviour is preserved if you
-  pass a lambda that takes just one argument, but this is deprecated and will be
-  removed in the next version of Circuitbox.
-  [dwaller](https://github.com/dwaller)
-
-### v0.10.1
-- [Documentation fix](https://github.com/yammer/circuitbox/pull/29) [chiefcll](https://github.com/chiefcll)
-- [Faraday middleware fix](https://github.com/yammer/circuitbox/pull/30) [chiefcll](https://github.com/chiefcll)
-
-### v0.10
-- configuration option for faraday middleware for what should be considered to open the circuit [enrico-scalavio](https://github.com/enrico-scalavino)
-- fix for issue 16, support of in_parallel requests in faraday middleware which were opening the circuit.
-- deprecate the __run_option__ `:storage_key`
-
-### v0.9
-- add `run!` method to raise exception on circuit open and service
-
-### v0.8
-- Everything prior to keeping the change log
-
 ## Installation
 
 Add this line to your application's Gemfile:

data/lib/circuitbox.rb

@@ -1,4 +1,5 @@
-require 'uri'
+# frozen_string_literal: true
+
 require 'logger'
 
 require_relative 'circuitbox/version'
@@ -12,12 +13,12 @@ class Circuitbox
   class << self
     include Configuration
 
-    def circuit(service_name, options)
+    def circuit(service_name, options, &block)
       circuit = (cached_circuits[service_name] ||= CircuitBreaker.new(service_name, options))
 
-      return circuit unless block_given?
+      return circuit unless block
 
-      circuit.run(circuitbox_exceptions: false) { yield }
+      circuit.run(circuitbox_exceptions: false, &block)
     end
   end
 end

data/lib/circuitbox/circuit_breaker.rb

@@ -7,13 +7,13 @@ class Circuitbox
     include LoggerMessages
 
     attr_reader :service, :circuit_options, :exceptions,
-                :logger, :circuit_store, :notifier, :time_class, :execution_timer
+                :logger, :circuit_store, :notifier, :time_class
 
     DEFAULTS = {
-      sleep_window:     90,
+      sleep_window: 90,
       volume_threshold: 5,
-      error_threshold:  50,
-      time_window:      60
+      error_threshold: 50,
+      time_window: 60
     }.freeze
 
     #
@@ -30,7 +30,6 @@ class Circuitbox
       @service = service.to_s
       @circuit_options = DEFAULTS.merge(options)
       @circuit_store   = options.fetch(:cache) { Circuitbox.default_circuit_store }
-      @execution_timer = options.fetch(:execution_timer) { Circuitbox.default_timer }
       @notifier = options.fetch(:notifier) { Circuitbox.default_notifier }
 
       if @circuit_options[:timeout_seconds]
@@ -39,7 +38,7 @@ class Circuitbox
       end
 
       @exceptions = options.fetch(:exceptions)
-      raise ArgumentError, 'exceptions need to be an array' unless @exceptions.is_a?(Array)
+      raise ArgumentError.new('exceptions need to be an array') unless @exceptions.is_a?(Array)
 
       @logger     = options.fetch(:logger) { Circuitbox.default_logger }
       @time_class = options.fetch(:time_class) { Time }
@@ -52,7 +51,7 @@ class Circuitbox
       value.is_a?(Proc) ? value.call : value
     end
 
-    def run(circuitbox_exceptions: true)
+    def run(circuitbox_exceptions: true, &block)
       if open?
         skipped!
         raise Circuitbox::OpenCircuitError.new(service) if circuitbox_exceptions
@@ -60,17 +59,15 @@ class Circuitbox
         logger.debug(circuit_running_message)
 
         begin
-          response = execution_timer.time(service, notifier, 'execution_time') do
-            yield
-          end
+          response = Timer.measure(service, notifier, 'runtime', &block)
 
           success!
-        rescue *exceptions => exception
+        rescue *exceptions => e
           # Other stores could raise an exception that circuitbox is asked to watch.
           # setting to nil keeps the same behavior as the previous defination of run.
           response = nil
           failure!
-          raise Circuitbox::ServiceFailureError.new(service, exception) if circuitbox_exceptions
+          raise Circuitbox::ServiceFailureError.new(service, e) if circuitbox_exceptions
         end
       end
 
@@ -83,7 +80,8 @@ class Circuitbox
 
     def error_rate(failures = failure_count, success = success_count)
       all_count = failures + success
-      return 0.0 unless all_count > 0
+      return 0.0 unless all_count.positive?
+
       (failures / all_count.to_f) * 100
     end
 
@@ -99,7 +97,7 @@ class Circuitbox
       circuit_store.delete(open_storage_key)
     end
 
-  private
+    private
 
     def should_open?
       failures = failure_count
@@ -206,12 +204,12 @@ class Circuitbox
     def check_sleep_window
       sleep_window = option_value(:sleep_window)
       time_window  = option_value(:time_window)
-      if sleep_window < time_window
-        warning_message = "sleep_window: #{sleep_window} is shorter than time_window: #{time_window}, "\
-                          "the error_rate would not be reset after a sleep."
-        notifier.notify_warning(service, warning_message)
-        warn("Circuit: #{service}, Warning: #{warning_message}")
-      end
+      return unless sleep_window < time_window
+
+      warning_message = "sleep_window: #{sleep_window} is shorter than time_window: #{time_window}, "\
+                        "the error_rate would not be reset after a sleep."
+      notifier.notify_warning(service, warning_message)
+      warn("Circuit: #{service}, Warning: #{warning_message}")
     end
 
     def stat_storage_key(event)

data/lib/circuitbox/configuration.rb

@@ -1,7 +1,7 @@
+# frozen_string_literal: true
+
 require_relative 'memory_store'
-require_relative 'timer/monotonic'
-require_relative 'timer/null'
-require_relative 'timer/simple'
+require_relative 'timer'
 require_relative 'notifier/active_support'
 require_relative 'notifier/null'
 
@@ -30,19 +30,15 @@ class Circuitbox
                             end
     end
 
-    def default_timer
-      @default_timer ||= Timer::Simple.new
-    end
-
     def default_logger
       @default_logger ||= if defined?(Rails)
                             Rails.logger
                           else
-                            Logger.new(STDOUT)
+                            Logger.new($stdout)
                           end
     end
 
-  private
+    private
 
     def cached_circuits
       @cached_circuits ||= {}

data/lib/circuitbox/errors/error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Circuitbox
   class Error < StandardError; end
 end

data/lib/circuitbox/errors/open_circuit_error.rb

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 class Circuitbox
   class OpenCircuitError < Circuitbox::Error
     attr_reader :service
 
     def initialize(service)
+      super()
       @service = service
     end
   end

data/lib/circuitbox/errors/service_failure_error.rb

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 class Circuitbox
   class ServiceFailureError < Circuitbox::Error
     attr_reader :service, :original
 
     def initialize(service, exception)
+      super()
       @service = service
       @original = exception
       # we copy over the original exceptions backtrace if there is one

data/lib/circuitbox/excon_middleware.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'excon'
 require 'circuitbox'
 
@@ -8,7 +10,7 @@ class Circuitbox
     DEFAULT_EXCEPTIONS = [
       Excon::Errors::Timeout,
       RequestFailed
-    ]
+    ].freeze
 
     class NullResponse < Excon::Response
       def initialize(response, exception)
@@ -26,7 +28,7 @@ class Circuitbox
 
     def initialize(stack, opts = {})
       @stack = stack
-      default_options = { open_circuit: lambda { |response| response[:status] >= 400 } }
+      default_options = { open_circuit: ->(response) { response[:status] >= 400 } }
       @opts = default_options.merge(opts)
       super(stack)
     end
@@ -35,8 +37,8 @@ class Circuitbox
       circuit(datum).run do
         raise RequestFailed
       end
-    rescue Circuitbox::Error => exception
-      circuit_open_value(datum, datum[:response], exception)
+    rescue Circuitbox::Error => e
+      circuit_open_value(datum, datum[:response], e)
     end
 
     def request_call(datum)
@@ -50,8 +52,8 @@ class Circuitbox
         raise RequestFailed if open_circuit?(datum[:response])
       end
       @stack.response_call(datum)
-    rescue Circuitbox::Error => exception
-      circuit_open_value(datum, datum[:response], exception)
+    rescue Circuitbox::Error => e
+      circuit_open_value(datum, datum[:response], e)
     end
 
     def identifier
@@ -93,9 +95,9 @@ class Circuitbox
     def default_value
       @default_value ||= begin
         default = opts.fetch(:default_value) do
-          lambda { |response, exception| NullResponse.new(response, exception) }
+          ->(response, exception) { NullResponse.new(response, exception) }
         end
-        default.respond_to?(:call) ? default : lambda { |*| default }
+        default.respond_to?(:call) ? default : ->(*) { default }
       end
     end
   end

data/lib/circuitbox/faraday_middleware.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'faraday'
 require 'circuitbox'
 
@@ -7,6 +9,7 @@ class Circuitbox
 
     class NullResponse < Faraday::Response
       attr_reader :original_response, :original_exception
+
       def initialize(response = nil, exception = nil)
         @original_response  = response
         @original_exception = exception
@@ -60,8 +63,8 @@ class Circuitbox
           raise RequestFailed if open_circuit?(service_response)
         end
       end
-    rescue Circuitbox::Error => ex
-      circuit_open_value(request_env, service_response, ex)
+    rescue Circuitbox::Error => e
+      circuit_open_value(request_env, service_response, e)
     end
 
     private

data/lib/circuitbox/memory_store.rb

@@ -51,7 +51,7 @@ class Circuitbox
       @mutex.synchronize { @store.delete(key) }
     end
 
-  private
+    private
 
     def fetch_container(key)
       current_time = current_second
@@ -73,6 +73,7 @@ class Circuitbox
     def fetch_value(key)
       container = fetch_container(key)
       return unless container
+
       container.value
     end
 

data/lib/circuitbox/memory_store/container.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'monotonic_time'
 
 class Circuitbox
@@ -13,11 +15,11 @@ class Circuitbox
       end
 
       def expired?
-        @expires_after > 0 && @expires_after < current_second
+        @expires_after.positive? && @expires_after < current_second
       end
 
       def expired_at?(clock_second)
-        @expires_after > 0 && @expires_after < clock_second
+        @expires_after.positive? && @expires_after < clock_second
       end
 
       def expires_after(seconds = 0)

data/lib/circuitbox/memory_store/monotonic_time.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Circuitbox
   class MemoryStore
     module MonotonicTime

data/lib/circuitbox/notifier/null.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Circuitbox
   class Notifier
     class Null

data/lib/circuitbox/timer.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+class Circuitbox
+  class Timer
+    class Monotonic
+      class << self
+        def supported?
+          defined?(Process::CLOCK_MONOTONIC)
+        end
+
+        def now
+          Process.clock_gettime(Process::CLOCK_MONOTONIC)
+        end
+      end
+    end
+
+    class Default
+      class << self
+        def supported?
+          true
+        end
+
+        def now
+          Time.now.to_f
+        end
+      end
+    end
+
+    class << self
+      def measure(service, notifier, metric_name)
+        before = now
+        result = yield
+        total_time = now - before
+        notifier.metric_gauge(service, metric_name, total_time)
+        result
+      end
+
+      private
+
+      if Monotonic.supported?
+        def now
+          Monotonic.now
+        end
+      else
+        def now
+          Default.now
+        end
+      end
+    end
+  end
+end

data/lib/circuitbox/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Circuitbox
-  VERSION = '2.0.0.pre3'.freeze
+  VERSION = '2.0.0.pre4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: circuitbox
 version: !ruby/object:Gem::Version
-  version: 2.0.0.pre3
+  version: 2.0.0.pre4
 platform: ruby
 authors:
 - Fahim Ferdous
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-28 00:00:00.000000000 Z
+date: 2021-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -45,7 +45,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: '0.8'
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -55,7 +55,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.15'
+        version: '0.8'
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -115,6 +115,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.7'
+- !ruby/object:Gem::Dependency
+  name: moneta
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
@@ -144,47 +158,89 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '13.0'
 - !ruby/object:Gem::Dependency
-  name: timecop
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: 1.8.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: 1.8.1
 - !ruby/object:Gem::Dependency
-  name: typhoeus
+  name: rubocop-minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.10.3
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.10.3
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.9.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.9.2
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.1
+- !ruby/object:Gem::Dependency
+  name: timecop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: moneta
+  name: typhoeus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.3'
 description: 
 email:
 - fahimfmf@gmail.com
@@ -208,14 +264,15 @@ files:
 - lib/circuitbox/memory_store/monotonic_time.rb
 - lib/circuitbox/notifier/active_support.rb
 - lib/circuitbox/notifier/null.rb
-- lib/circuitbox/timer/monotonic.rb
-- lib/circuitbox/timer/null.rb
-- lib/circuitbox/timer/simple.rb
+- lib/circuitbox/timer.rb
 - lib/circuitbox/version.rb
 homepage: https://github.com/yammer/circuitbox
 licenses:
 - Apache-2.0
-metadata: {}
+metadata:
+  bug_tracker_uri: https://github.com/yammer/circuitbox/issues
+  changelog_uri: https://github.com/yammer/circuitbox/blob/main/CHANGELOG.md
+  source_code_uri: https://github.com/yammer/circuitbox
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -224,15 +281,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.3
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: A robust circuit breaker that manages failing external services.

data/lib/circuitbox/timer/monotonic.rb

@@ -1,17 +0,0 @@
-class Circuitbox
-  class Timer
-    class Monotonic
-      def initialize(time_unit = :millisecond)
-        @time_unit = time_unit
-      end
-
-      def time(service, notifier, metric_name)
-        before = Process.clock_gettime(Process::CLOCK_MONOTONIC, @time_unit)
-        result = yield
-        total_time = Process.clock_gettime(Process::CLOCK_MONOTONIC, @time_unit) - before
-        notifier.metric_gauge(service, metric_name, total_time)
-        result
-      end
-    end
-  end
-end

data/lib/circuitbox/timer/null.rb

@@ -1,9 +0,0 @@
-class Circuitbox
-  class Timer
-    class Null
-      def time(_service, _notifier, _metric_name)
-        yield
-      end
-    end
-  end
-end

data/lib/circuitbox/timer/simple.rb

@@ -1,13 +0,0 @@
-class Circuitbox
-  class Timer
-    class Simple
-      def time(service, notifier, metric_name)
-        before = Time.now.to_f
-        result = yield
-        total_time = Time.now.to_f - before
-        notifier.metric_gauge(service, metric_name, total_time)
-        result
-      end
-    end
-  end
-end