analytics-ruby 2.2.3.pre → 2.2.4.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b487a2b95a07ae396262ebf9af60f94b09152568
-  data.tar.gz: 577c4474658bfcc5a40baf1dc55abd9c918f3728
+SHA256:
+  metadata.gz: 276383d9b3932f6d39493e74cf48ec05fb6805050c4031acd77a245e9b6e6d4e
+  data.tar.gz: ebe9076f70b26805478f40a3c45bbf5c13bdd8de6083081651cbc1a3476f6e4b
 SHA512:
-  metadata.gz: d6b249cff66b4b304ef19bab340abc9cbc83e5ff0b6007a885dd22409f60c415a11b33dba43a8a99b3032122b822270979dd16fc398afdebac152f8b67e1e66e
-  data.tar.gz: ae3a5effa3de14b3e291358d0ca0a66832fe56639c77c8c07d7b6ea5b2f00a5da1568bef10666f1e5c76659eeb99a04ec746be596b76ba5109a33e88b1856e5b
+  metadata.gz: 9d5ba302e4d25b5155a8d273d8e9d406bce39e5bb906f205afd58c86fcd816f90512fb7117b6a19c98a5b7bde3d1b8c8e8158465a3c04154e6bfee9ca406602f
+  data.tar.gz: 7b935c07754a56217f8735161bab41920bcd4b8bea94359c8a0d5cd04e0b2bb2d63ff3aff478eb72f6978922b63c00d22027f24949d0badef6e7b659b573f5c6

data/History.md

@@ -1,3 +1,19 @@
+2.2.4.pre / 2018-02-04
+==================
+
+  * [Fix](https://github.com/segmentio/analytics-ruby/pull/147): Prevent 'batch
+    size exceeded' errors by automatically batching
+    items according to size
+  * [Performance](https://github.com/segmentio/analytics-ruby/pull/149): Reuse
+    TCP connections
+  * [Improvement](https://github.com/segmentio/analytics-ruby/pull/145): Emit logs
+    when in-memory queue is full
+  * [Improvement](https://github.com/segmentio/analytics-ruby/pull/143): Emit logs
+    when messages exceed maximum allowed size
+  * [Improvement](https://github.com/segmentio/analytics-ruby/pull/134): Add
+    exponential backoff to retries
+  * [Improvement](https://github.com/segmentio/analytics-ruby/pull/132): Handle
+    HTTP status code failure appropriately
 
 2.2.3.pre / 2017-09-14
 ==================

data/Makefile

@@ -1,8 +1,17 @@
-
-test:
-	rake spec
-
-build:
-	gem build ./analytics-ruby.gemspec
-
-.PHONY: test build
+
+# Install any tools required to build this library, e.g. Ruby, Bundler etc.
+bootstrap:
+	brew install ruby
+	gem install bundler
+
+# Install any library dependencies.
+dependencies:
+	bundle install --verbose
+
+# Run all tests and checks (including linters).
+check:
+	bundle exec rake
+
+# Compile the code and produce any binaries where applicable.
+build:
+	gem build ./analytics-ruby.gemspec

data/README.md

@@ -9,11 +9,11 @@ analytics-ruby is a ruby client for [Segment](https://segment.com)
 
 Into Gemfile from rubygems.org:
 ```ruby
-gem 'analytics-ruby', require: "segment"
+gem 'analytics-ruby'
 ```
 
 Into environment gems from rubygems.org:
-```ruby
+```
 gem install 'analytics-ruby'
 ```
 

data/RELEASING.md

@@ -5,6 +5,5 @@ Releasing
  2. Bump version in [`version.rb`](https://github.com/segmentio/analytics-ruby/blob/master/lib/segment/analytics/version.rb).
  3. Update [`History.md`](https://github.com/segmentio/analytics-ruby/blob/master/History.md).
  4. Commit and tag `git commit -am "Release {version}" && git tag -a {version} -m "Version {version}"`.
- 5. Build the gem with the tagged version `make build`.
- 6. Upload to RubyGems with `gem push analytics-ruby-{version}.gem`.
- 7. Upload to Github with `git push -u origin master && git push --tags`.
+ 5. Upload to Github with `git push -u origin master && git push --tags`.
+The tagged commit will be pushed to RubyGems via Travis.

data/Rakefile

@@ -1,7 +1,23 @@
 require 'rspec/core/rake_task'
 
+default_tasks = []
+
 RSpec::Core::RakeTask.new(:spec) do |spec|
   spec.pattern = 'spec/**/*_spec.rb'
+  spec.rspec_opts = "--tag ~e2e" if ENV["RUN_E2E_TESTS"] != "true"
+end
+
+default_tasks << :spec
+
+# Rubocop doesn't support < 2.1
+if RUBY_VERSION >= "2.1"
+  require 'rubocop/rake_task'
+
+  RuboCop::RakeTask.new(:rubocop) do |task|
+    task.patterns = ['lib/**/*.rb','spec/**/*.rb',]
+  end
+
+  default_tasks << :rubocop
 end
 
-task :default => :spec
+task :default => default_tasks

data/analytics-ruby.gemspec

@@ -19,7 +19,15 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'commander', '~> 4.4'
 
   spec.add_development_dependency 'rake', '~> 10.3'
-  spec.add_development_dependency 'rspec', '~> 2.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'tzinfo', '1.2.1'
-  spec.add_development_dependency 'activesupport', '>= 3.0.0', '<4.0.0'
+  spec.add_development_dependency 'activesupport', '~> 4.1.11'
+  spec.add_development_dependency 'faraday', '~> 0.13'
+  spec.add_development_dependency 'pmap', '~> 1.1'
+
+  if RUBY_VERSION >= "2.1"
+    spec.add_development_dependency 'rubocop', '~> 0.51.0'
+  end
+
+  spec.add_development_dependency 'codecov', '~> 0.1.4'
 end

data/codecov.yml

@@ -0,0 +1,2 @@
+ignore:
+  - "spec/**/*"

data/lib/analytics-ruby.rb

@@ -0,0 +1 @@
+require 'segment'

data/lib/segment/analytics.rb

@@ -9,12 +9,19 @@ require 'segment/analytics/logging'
 
 module Segment
   class Analytics
-    def initialize options = {}
+    # Initializes a new instance of {Segment::Analytics::Client}, to which all
+    # method calls are proxied.
+    #
+    # @param options includes options that are passed down to
+    #   {Segment::Analytics::Client#initialize}
+    # @option options [Boolean] :stub (false) If true, requests don't hit the
+    #   server and are stubbed to be successful.
+    def initialize(options = {})
       Request.stub = options[:stub] if options.has_key?(:stub)
       @client = Segment::Analytics::Client.new options
     end
 
-    def method_missing message, *args, &block
+    def method_missing(message, *args, &block)
       if @client.respond_to? message
         @client.send message, *args, &block
       else

data/lib/segment/analytics/backoff_policy.rb

@@ -0,0 +1,49 @@
+require 'segment/analytics/defaults'
+
+module Segment
+  class Analytics
+    class BackoffPolicy
+      include Segment::Analytics::Defaults::BackoffPolicy
+
+      # @param [Hash] opts
+      # @option opts [Numeric] :min_timeout_ms The minimum backoff timeout
+      # @option opts [Numeric] :max_timeout_ms The maximum backoff timeout
+      # @option opts [Numeric] :multiplier The value to multiply the current
+      #   interval with for each retry attempt
+      # @option opts [Numeric] :randomization_factor The randomization factor
+      #   to use to create a range around the retry interval
+      def initialize(opts = {})
+        @min_timeout_ms = opts[:min_timeout_ms] || MIN_TIMEOUT_MS
+        @max_timeout_ms = opts[:max_timeout_ms] || MAX_TIMEOUT_MS
+        @multiplier = opts[:multiplier] || MULTIPLIER
+        @randomization_factor = opts[:randomization_factor] || RANDOMIZATION_FACTOR
+
+        @attempts = 0
+      end
+
+      # @return [Numeric] the next backoff interval, in milliseconds.
+      def next_interval
+        interval = @min_timeout_ms * (@multiplier**@attempts)
+        interval = add_jitter(interval, @randomization_factor)
+
+        @attempts += 1
+
+        [interval, @max_timeout_ms].min
+      end
+
+      private
+
+      def add_jitter(base, randomization_factor)
+        random_number = rand
+        max_deviation = base * randomization_factor
+        deviation = random_number * max_deviation
+
+        if random_number < 0.5
+          base - deviation
+        else
+          base + deviation
+        end
+      end
+    end
+  end
+end

data/lib/segment/analytics/client.rb

@@ -1,39 +1,41 @@
 require 'thread'
 require 'time'
+
+require 'segment/analytics/defaults'
+require 'segment/analytics/logging'
 require 'segment/analytics/utils'
 require 'segment/analytics/worker'
-require 'segment/analytics/defaults'
 
 module Segment
   class Analytics
     class Client
       include Segment::Analytics::Utils
+      include Segment::Analytics::Logging
 
-      # public: Creates a new client
-      #
-      # attrs - Hash
-      #           :write_key         - String of your project's write_key
-      #           :max_queue_size - Fixnum of the max calls to remain queued (optional)
-      #           :on_error       - Proc which handles error calls from the API
-      def initialize attrs = {}
-        symbolize_keys! attrs
+      # @param [Hash] opts
+      # @option opts [String] :write_key Your project's write_key
+      # @option opts [FixNum] :max_queue_size Maximum number of calls to be
+      #   remain queued.
+      # @option opts [Proc] :on_error Handles error calls from the API.
+      def initialize(opts = {})
+        symbolize_keys!(opts)
 
         @queue = Queue.new
-        @write_key = attrs[:write_key]
-        @max_queue_size = attrs[:max_queue_size] || Defaults::Queue::MAX_SIZE
-        @options = attrs
+        @write_key = opts[:write_key]
+        @max_queue_size = opts[:max_queue_size] || Defaults::Queue::MAX_SIZE
+        @options = opts
         @worker_mutex = Mutex.new
-        @worker = Worker.new @queue, @write_key, @options
+        @worker = Worker.new(@queue, @write_key, @options)
 
         check_write_key!
 
         at_exit { @worker_thread && @worker_thread[:should_exit] = true }
       end
 
-      # public: Synchronously waits until the worker has flushed the queue.
-      #         Use only for scripts which are not long-running, and will
-      #         specifically exit
+      # Synchronously waits until the worker has flushed the queue.
       #
+      # Use only for scripts which are not long-running, and will specifically
+      # exit
       def flush
         while !@queue.empty? || @worker.is_requesting?
           ensure_worker_running
@@ -41,19 +43,26 @@ module Segment
         end
       end
 
-      # public: Tracks an event
+      # Tracks an event
       #
-      # attrs - Hash
-      #           :anonymous_id - String of the user's id when you don't know who they are yet. (optional but you must provide either an anonymous_id or user_id. See: https://segment.io/docs/tracking - api/track/#user - id)
-      #           :context      - Hash of context. (optional)
-      #           :event        - String of event name.
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :properties   - Hash of event properties. (optional)
-      #           :timestamp    - Time of when the event occurred. (optional)
-      #           :user_id      - String of the user id.
-      #           :message_id   - String of the message id that uniquely identified a message across the API. (optional)
-      def track attrs
+      # @see https://segment.com/docs/sources/server/ruby/#track
+      #
+      # @param [Hash] attrs
+      # @option attrs [String] :anonymous_id ID for a user when you don't know
+      #   who they are yet. (optional but you must provide either an
+      #   `anonymous_id` or `user_id`)
+      # @option attrs [Hash] :context ({})
+      # @option attrs [String] :event Event name
+      # @option attrs [Hash] :integrations What integrations this event
+      #   goes to (optional)
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [Hash] :properties Event properties (optional)
+      # @option attrs [Time] :timestamp When the event occurred (optional)
+      # @option attrs [String] :user_id The ID for this user in your database
+      #   (optional but you must provide either an `anonymous_id` or `user_id`)
+      # @option attrs [String] :message_id ID that uniquely
+      #   identifies a message across the API. (optional)
+      def track(attrs)
         symbolize_keys! attrs
         check_user_id! attrs
 
@@ -66,10 +75,10 @@ module Segment
         check_timestamp! timestamp
 
         if event.nil? || event.empty?
-          fail ArgumentError, 'Must supply event as a non-empty string'
+          raise ArgumentError, 'Must supply event as a non-empty string'
         end
 
-        fail ArgumentError, 'Properties must be a Hash' unless properties.is_a? Hash
+        raise ArgumentError, 'Properties must be a Hash' unless properties.is_a? Hash
         isoify_dates! properties
 
         add_context context
@@ -88,18 +97,25 @@ module Segment
         })
       end
 
-      # public: Identifies a user
+      # Identifies a user
+      #
+      # @see https://segment.com/docs/sources/server/ruby/#identify
       #
-      # attrs - Hash
-      #           :anonymous_id - String of the user's id when you don't know who they are yet. (optional but you must provide either an anonymous_id or user_id. See: https://segment.io/docs/tracking - api/track/#user - id)
-      #           :context      - Hash of context. (optional)
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :timestamp    - Time of when the event occurred. (optional)
-      #           :traits       - Hash of user traits. (optional)
-      #           :user_id      - String of the user id
-      #           :message_id   - String of the message id that uniquely identified a message across the API. (optional)
-      def identify attrs
+      # @param [Hash] attrs
+      # @option attrs [String] :anonymous_id ID for a user when you don't know
+      #   who they are yet. (optional but you must provide either an
+      #   `anonymous_id` or `user_id`)
+      # @option attrs [Hash] :context ({})
+      # @option attrs [Hash] :integrations What integrations this event
+      #   goes to (optional)
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [Time] :timestamp When the event occurred (optional)
+      # @option attrs [Hash] :traits User traits (optional)
+      # @option attrs [String] :user_id The ID for this user in your database
+      #   (optional but you must provide either an `anonymous_id` or `user_id`)
+      # @option attrs [String] :message_id ID that uniquely identifies a
+      #   message across the API. (optional)
+      def identify(attrs)
         symbolize_keys! attrs
         check_user_id! attrs
 
@@ -110,7 +126,7 @@ module Segment
 
         check_timestamp! timestamp
 
-        fail ArgumentError, 'Must supply traits as a hash' unless traits.is_a? Hash
+        raise ArgumentError, 'Must supply traits as a hash' unless traits.is_a? Hash
         isoify_dates! traits
 
         add_context context
@@ -128,16 +144,20 @@ module Segment
         })
       end
 
-      # public: Aliases a user from one id to another
+      # Aliases a user from one id to another
       #
-      # attrs - Hash
-      #           :context     - Hash of context (optional)
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :previous_id - String of the id to alias from
-      #           :timestamp   - Time of when the alias occured (optional)
-      #           :user_id     - String of the id to alias to
-      #           :message_id   - String of the message id that uniquely identified a message across the API. (optional)
+      # @see https://segment.com/docs/sources/server/ruby/#alias
+      #
+      # @param [Hash] attrs
+      # @option attrs [Hash] :context ({})
+      # @option attrs [Hash] :integrations What integrations this must be
+      #   sent to (optional)
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [String] :previous_id The ID to alias from
+      # @option attrs [Time] :timestamp When the alias occurred (optional)
+      # @option attrs [String] :user_id The ID to alias to
+      # @option attrs [String] :message_id ID that uniquely identifies a
+      #   message across the API. (optional)
       def alias(attrs)
         symbolize_keys! attrs
 
@@ -164,16 +184,24 @@ module Segment
         })
       end
 
-      # public: Associates a user identity with a group.
+      # Associates a user identity with a group.
+      #
+      # @see https://segment.com/docs/sources/server/ruby/#group
       #
-      # attrs - Hash
-      #           :context      - Hash of context (optional)
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :previous_id  - String of the id to alias from
-      #           :timestamp    - Time of when the alias occured (optional)
-      #           :user_id      - String of the id to alias to
-      #           :message_id   - String of the message id that uniquely identified a message across the API. (optional)
+      # @param [Hash] attrs
+      # @option attrs [String] :anonymous_id ID for a user when you don't know
+      #   who they are yet. (optional but you must provide either an
+      #   `anonymous_id` or `user_id`)
+      # @option attrs [Hash] :context ({})
+      # @option attrs [String] :group_id The ID of the group
+      # @option attrs [Hash] :integrations What integrations this event
+      #   goes to (optional)
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [Time] :timestamp When the event occurred (optional)
+      # @option attrs [String] :user_id The ID for the user that is part of
+      #   the group
+      # @option attrs [String] :message_id ID that uniquely identifies a
+      #   message across the API. (optional)
       def group(attrs)
         symbolize_keys! attrs
         check_user_id! attrs
@@ -185,7 +213,7 @@ module Segment
         context = attrs[:context] || {}
         message_id = attrs[:message_id].to_s if attrs[:message_id]
 
-        fail ArgumentError, '.traits must be a hash' unless traits.is_a? Hash
+        raise ArgumentError, '.traits must be a hash' unless traits.is_a? Hash
         isoify_dates! traits
 
         check_presence! group_id, 'group_id'
@@ -205,19 +233,25 @@ module Segment
         })
       end
 
-      # public: Records a page view
+      # Records a page view
       #
-      # attrs - Hash
-      #           :anonymous_id - String of the user's id when you don't know who they are yet. (optional but you must provide either an anonymous_id or user_id. See: https://segment.io/docs/tracking - api/track/#user - id)
-      #           :category     - String of the page category (optional)
-      #           :context      - Hash of context (optional)
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :name         - String name of the page
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :properties   - Hash of page properties (optional)
-      #           :timestamp    - Time of when the pageview occured (optional)
-      #           :user_id      - String of the id to alias from
-      #           :message_id   - String of the message id that uniquely identified a message across the API. (optional)
+      # @see https://segment.com/docs/sources/server/ruby/#page
+      #
+      # @param [Hash] attrs
+      # @option attrs [String] :anonymous_id ID for a user when you don't know
+      #   who they are yet. (optional but you must provide either an
+      #   `anonymous_id` or `user_id`)
+      # @option attrs [String] :category The page category (optional)
+      # @option attrs [Hash] :context ({})
+      # @option attrs [Hash] :integrations What integrations this event
+      #   goes to (optional)
+      # @option attrs [String] :name Name of the page
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [Hash] :properties Page properties (optional)
+      # @option attrs [Time] :timestamp When the pageview occurred (optional)
+      # @option attrs [String] :user_id The ID of the user viewing the page
+      # @option attrs [String] :message_id ID that uniquely identifies a
+      #   message across the API. (optional)
       def page(attrs)
         symbolize_keys! attrs
         check_user_id! attrs
@@ -228,7 +262,7 @@ module Segment
         context = attrs[:context] || {}
         message_id = attrs[:message_id].to_s if attrs[:message_id]
 
-        fail ArgumentError, '.properties must be a hash' unless properties.is_a? Hash
+        raise ArgumentError, '.properties must be a hash' unless properties.is_a? Hash
         isoify_dates! properties
 
         check_timestamp! timestamp
@@ -248,18 +282,24 @@ module Segment
           :type => 'page'
         })
       end
-      # public: Records a screen view (for a mobile app)
+
+      # Records a screen view (for a mobile app)
       #
-      # attrs - Hash
-      #           :anonymous_id - String of the user's id when you don't know who they are yet. (optional but you must provide either an anonymous_id or user_id. See: https://segment.io/docs/tracking - api/track/#user - id)
-      #           :category     - String screen category (optional)
-      #           :context      - Hash of context (optional)
-      #           :integrations - Hash specifying what integrations this event goes to. (optional)
-      #           :name         - String name of the screen
-      #           :options      - Hash specifying options such as user traits. (optional)
-      #           :properties   - Hash of screen properties (optional)
-      #           :timestamp    - Time of when the screen occured (optional)
-      #           :user_id      - String of the id to alias from
+      # @param [Hash] attrs
+      # @option attrs [String] :anonymous_id ID for a user when you don't know
+      #   who they are yet. (optional but you must provide either an
+      #   `anonymous_id` or `user_id`)
+      # @option attrs [String] :category The screen category (optional)
+      # @option attrs [Hash] :context ({})
+      # @option attrs [Hash] :integrations What integrations this event
+      #   goes to (optional)
+      # @option attrs [String] :name Name of the screen
+      # @option attrs [Hash] :options Options such as user traits (optional)
+      # @option attrs [Hash] :properties Page properties (optional)
+      # @option attrs [Time] :timestamp When the pageview occurred (optional)
+      # @option attrs [String] :user_id The ID of the user viewing the screen
+      # @option attrs [String] :message_id ID that uniquely identifies a
+      #   message across the API. (optional)
       def screen(attrs)
         symbolize_keys! attrs
         check_user_id! attrs
@@ -270,7 +310,7 @@ module Segment
         context = attrs[:context] || {}
         message_id = attrs[:message_id].to_s if attrs[:message_id]
 
-        fail ArgumentError, '.properties must be a hash' unless properties.is_a? Hash
+        raise ArgumentError, '.properties must be a hash' unless properties.is_a? Hash
         isoify_dates! properties
 
         check_timestamp! timestamp
@@ -291,9 +331,7 @@ module Segment
         })
       end
 
-      # public: Returns the number of queued messages
-      #
-      # returns Fixnum of messages in the queue
+      # @return [Fixnum] number of messages in the queue
       def queued_messages
         @queue.length
       end
@@ -306,11 +344,20 @@ module Segment
       def enqueue(action)
         # add our request id for tracing purposes
         action[:messageId] ||= uid
-        unless queue_full = @queue.length >= @max_queue_size
-          ensure_worker_running
+
+        if @queue.length < @max_queue_size
           @queue << action
+          ensure_worker_running
+
+          true
+        else
+          logger.warn(
+            'Queue is full, dropping events. The :max_queue_size ' \
+            'configuration parameter can be increased to prevent this from ' \
+            'happening.'
+          )
+          false
         end
-        !queue_full
       end
 
       # private: Ensures that a string is non-empty
@@ -320,7 +367,7 @@ module Segment
       #
       def check_presence!(obj, name)
         if obj.nil? || (obj.is_a?(String) && obj.empty?)
-          fail ArgumentError, "#{name} must be given"
+          raise ArgumentError, "#{name} must be given"
         end
       end
 
@@ -328,20 +375,20 @@ module Segment
       #
       # context - Hash of call context
       def add_context(context)
-        context[:library] =  { :name => "analytics-ruby", :version => Segment::Analytics::VERSION.to_s }
+        context[:library] = { :name => 'analytics-ruby', :version => Segment::Analytics::VERSION.to_s }
       end
 
       # private: Checks that the write_key is properly initialized
       def check_write_key!
-        fail ArgumentError, 'Write key must be initialized' if @write_key.nil?
+        raise ArgumentError, 'Write key must be initialized' if @write_key.nil?
       end
 
       # private: Checks the timstamp option to make sure it is a Time.
       def check_timestamp!(timestamp)
-        fail ArgumentError, 'Timestamp must be a Time' unless timestamp.is_a? Time
+        raise ArgumentError, 'Timestamp must be a Time' unless timestamp.is_a? Time
       end
 
-      def event attrs
+      def event(attrs)
         symbolize_keys! attrs
 
         {
@@ -354,8 +401,10 @@ module Segment
         }
       end
 
-      def check_user_id! attrs
-        fail ArgumentError, 'Must supply either user_id or anonymous_id' unless attrs[:user_id] || attrs[:anonymous_id]
+      def check_user_id!(attrs)
+        unless attrs[:user_id] || attrs[:anonymous_id]
+          raise ArgumentError, 'Must supply either user_id or anonymous_id'
+        end
       end
 
       def ensure_worker_running