-
Notifications
You must be signed in to change notification settings - Fork 85
Error rate circuit breaker #264
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 3 commits
d5fb106
bad8a75
73a839d
08e851f
956211c
8a6c96f
4adee5e
4cfcf59
35b47d1
e2255d7
1bdff25
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,166 @@ | ||
| module Semian | ||
| class ErrorRateCircuitBreaker #:nodoc: | ||
| extend Forwardable | ||
|
|
||
| def_delegators :@state, :closed?, :open?, :half_open? | ||
|
|
||
| attr_reader :name, :half_open_resource_timeout, :error_timeout, :state, :last_error, :error_percent_threshold, | ||
| :request_volume_threshold, :success_count_threshold | ||
|
|
||
| def initialize(name, exceptions:, error_percent_threshold:, error_timeout:, window_size:, | ||
| request_volume_threshold:, success_threshold:, implementation:, half_open_resource_timeout: nil) | ||
damianthe marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| raise 'error_threshold_percent should be between 0.0 and 1.0 exclusive' unless (0.0001...1.0).cover?(error_percent_threshold) | ||
damianthe marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| @name = name.to_sym | ||
| @error_timeout = error_timeout | ||
| @exceptions = exceptions | ||
| @half_open_resource_timeout = half_open_resource_timeout | ||
| @error_percent_threshold = error_percent_threshold | ||
| @last_error_time = nil | ||
| @request_volume_threshold = request_volume_threshold | ||
| @success_count_threshold = success_threshold | ||
|
|
||
| @window = implementation::TimeSlidingWindow.new(window_size) | ||
| @state = implementation::State.new | ||
|
|
||
| reset | ||
| end | ||
|
|
||
| def acquire(resource = nil, &block) | ||
| return yield if disabled? | ||
| transition_to_half_open if transition_to_half_open? | ||
|
|
||
| raise OpenCircuitError unless request_allowed? | ||
|
|
||
| result = nil | ||
| begin | ||
| result = maybe_with_half_open_resource_timeout(resource, &block) | ||
| rescue *@exceptions => error | ||
| if !error.respond_to?(:marks_semian_circuits?) || error.marks_semian_circuits? | ||
| mark_failed(error) | ||
| end | ||
| raise error | ||
| else | ||
| mark_success | ||
| end | ||
| result | ||
| end | ||
|
|
||
| def transition_to_half_open? | ||
| open? && error_timeout_expired? && !half_open? | ||
| end | ||
|
|
||
| def request_allowed? | ||
| closed? || half_open? || transition_to_half_open? | ||
| end | ||
|
|
||
| def mark_failed(error) | ||
| push_error(error) | ||
| if closed? | ||
| transition_to_open if error_threshold_reached? | ||
damianthe marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| elsif half_open? | ||
| transition_to_open | ||
| end | ||
| end | ||
|
|
||
| def mark_success | ||
| @window << true | ||
| return unless half_open? | ||
| transition_to_close if success_threshold_reached? | ||
| end | ||
|
|
||
| def reset | ||
| @last_error_time = nil | ||
| @window.clear | ||
| transition_to_close | ||
| end | ||
|
|
||
| def destroy | ||
| @state.destroy | ||
| end | ||
|
|
||
| def in_use? | ||
| return false if error_timeout_expired? | ||
| @window.count(false) > 0 | ||
| end | ||
|
|
||
| private | ||
|
|
||
| def current_time | ||
| Process.clock_gettime(Process::CLOCK_MONOTONIC) | ||
| end | ||
|
|
||
| def transition_to_close | ||
| notify_state_transition(:closed) | ||
| log_state_transition(:closed) | ||
| @state.close! | ||
| @window.clear | ||
| end | ||
|
|
||
| def transition_to_open | ||
| notify_state_transition(:open) | ||
| log_state_transition(:open) | ||
| @state.open! | ||
| end | ||
|
|
||
| def transition_to_half_open | ||
| notify_state_transition(:half_open) | ||
| log_state_transition(:half_open) | ||
| @state.half_open! | ||
| @window.clear | ||
| end | ||
|
|
||
| def success_threshold_reached? | ||
| @window.count(true) >= @success_count_threshold | ||
| end | ||
|
|
||
| def error_threshold_reached? | ||
| return false if @window.empty? or @window.length < @request_volume_threshold | ||
|
||
| @window.count(false).to_f / @window.length.to_f >= @error_percent_threshold | ||
| end | ||
|
|
||
| def error_timeout_expired? | ||
| return false unless @last_error_time | ||
| current_time - @last_error_time >= @error_timeout | ||
| end | ||
|
|
||
| def push_error(error) | ||
| @last_error = error | ||
| @last_error_time = current_time | ||
| @window << false | ||
| end | ||
|
|
||
| def log_state_transition(new_state) | ||
| return if @state.nil? || new_state == @state.value | ||
|
|
||
| str = "[#{self.class.name}] State transition from #{@state.value} to #{new_state}." | ||
| str << " success_count=#{@window.count(true)} error_count=#{@window.count(false)}" | ||
| str << " success_count_threshold=#{@success_count_threshold} error_count_percent=#{@error_percent_threshold}" | ||
| str << " error_timeout=#{@error_timeout} error_last_at=\"#{@last_error_time}\"" | ||
| str << " name=\"#{@name}\"" | ||
| Semian.logger.info(str) | ||
| end | ||
|
|
||
| def notify_state_transition(new_state) | ||
| Semian.notify(:state_change, self, nil, nil, state: new_state) | ||
| end | ||
|
|
||
| def disabled? | ||
| ENV['SEMIAN_CIRCUIT_BREAKER_DISABLED'] || ENV['SEMIAN_DISABLED'] | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Requires rethinking of changes in master branch. Idea to not use ENV inside business logic, only during the configuration phase. |
||
| end | ||
|
|
||
| def maybe_with_half_open_resource_timeout(resource, &block) | ||
| result = | ||
| if half_open? && @half_open_resource_timeout && resource.respond_to?(:with_resource_timeout) | ||
|
||
| resource.with_resource_timeout(@half_open_resource_timeout) do | ||
| block.call | ||
| end | ||
| else | ||
| block.call | ||
| end | ||
|
|
||
| result | ||
| end | ||
| end | ||
| end | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,88 @@ | ||
| require 'thread' | ||
|
|
||
| module Semian | ||
| module Simple | ||
|
|
||
| class TimeSlidingWindow #:nodoc: | ||
| extend Forwardable | ||
|
|
||
| def_delegators :@window, :size, :empty?, :length | ||
| attr_reader :time_window_millis | ||
|
|
||
| Pair = Struct.new(:head, :tail) | ||
|
|
||
| # A sliding window is a structure that stores the most recent entries that were pushed within the last slice of time | ||
| def initialize(time_window) | ||
| @time_window_millis = time_window * 1000 | ||
| @window = [] | ||
| end | ||
|
|
||
| def count(arg) | ||
| vals = @window.map { |pair| pair.tail} | ||
| vals.count(arg) | ||
| end | ||
|
|
||
| def push(value) | ||
| remove_old # make room | ||
| @window << Pair.new(current_time, value) | ||
| self | ||
| end | ||
|
|
||
| alias_method :<<, :push | ||
|
|
||
| def clear | ||
| @window.clear | ||
| self | ||
| end | ||
|
|
||
| def last | ||
| @window.last&.tail | ||
| end | ||
|
|
||
| def remove_old | ||
| midtime = current_time - time_window_millis | ||
| # special case, everything is too old | ||
| @window.clear if [email protected]? and @window.last.head < midtime | ||
| # otherwise we find the index position where the cutoff is | ||
| idx = ([email protected]).bsearch { |n| @window[n].head >= midtime } | ||
damianthe marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| @window.slice!(0, idx) if idx | ||
| end | ||
|
|
||
| alias_method :destroy, :clear | ||
|
|
||
| private | ||
|
|
||
| def current_time | ||
| Process.clock_gettime(Process::CLOCK_MONOTONIC, :float_millisecond) | ||
| end | ||
| end | ||
| end | ||
|
|
||
| module ThreadSafe | ||
| class TimeSlidingWindow < Simple::TimeSlidingWindow | ||
| 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 count(*) | ||
| @lock.synchronize { super } | ||
| end | ||
|
|
||
| def remove_old | ||
| @lock.synchronize { super } | ||
| end | ||
|
|
||
| def push(*) | ||
| @lock.synchronize { super } | ||
| end | ||
| end | ||
| end | ||
| end | ||
Uh oh!
There was an error while loading. Please reload this page.