statful-client 2.0.1 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 46498a341544954211b78c9d1f90141be0bbbf49
-  data.tar.gz: 94f6d45010aafa42ba164fffee13a5e7fdd0af09
+  metadata.gz: 9d4790086f7421d9a53045ba31a4da10bb8b2ca7
+  data.tar.gz: 67a6103ba572fb7380fa0e6f79357840462deea0
 SHA512:
-  metadata.gz: 9097c70c4dacb2e54769c4b1ea97bccdf3edfe47793c04db1f958a042e41f31778eb0f8685aecd41e61bbc1ac34b594f2c5e430677703acba93de1103bfb322a
-  data.tar.gz: 00b54ada9449590172401c7bad55619436672c73a1a73e6d1dc02a598db2bd542f4f3d7d98f8fd6ca50a8b639cdc881396f1357adaddf089b8df2b6a7f65b685
+  metadata.gz: 0319bdf271d17dffeda24fc1b1cdcd743d04e41e4e3a9a4978e31b2a6e8783caae7ac2ea2583b8fae223e44171e3042573e634724edd0f97148294d9430562cf
+  data.tar.gz: c7846b4405e5a3599a783ef07865c305ebd25d9fe7f74bc8df0178e8740058c4fa4f732fc1cab6d9138f7a5b1519a1e7675ac0e6b3d442863e3774c243572d1b

data/lib/client.rb

@@ -1,6 +1,7 @@
 require 'socket'
 require 'delegate'
 require 'net/http'
+require 'concurrent'
 
 # Statful Client Instance
 #
@@ -27,6 +28,7 @@ class StatfulClient
   # @option config [Integer] :sample_rate Global sample rate (as a percentage), between: (1-100)
   # @option config [String] :namespace Global default namespace
   # @option config [Integer] :flush_size Buffer flush upper size limit
+  # @option config [Integer] :thread_pool_size Thread pool upper size limit
   # @return [Object] The Statful client
   def initialize(config = {})
     user_config = MyHash[config].symbolize_keys
@@ -50,12 +52,16 @@ class StatfulClient
       :tags => {},
       :sample_rate => 100,
       :namespace => 'application',
-      :flush_size => 5
+      :flush_size => 5,
+      :thread_pool_size => 5
     }
 
     @config = default_config.merge(user_config)
     @logger = @config[:logger]
-    @buffer = []
+
+    @buffer = MyQueue.new
+    @pool = Concurrent::FixedThreadPool.new(@config[:thread_pool_size])
+
     @http = Net::HTTP.new(@config[:host], @config[:port])
     @http.use_ssl = true # must enforce use of ssl, otherwise it will raise EOFError: end of file reached
 
@@ -230,7 +236,7 @@ class StatfulClient
   # @private
   def flush
     unless @buffer.empty?
-      message = @buffer.join('\n')
+      message = @buffer.to_a.join('\n')
 
       # Handle socket errors by just logging if we have a logger instantiated
       # We could eventually save the buffer state but that would require us to manage the buffer size etc.
@@ -275,10 +281,18 @@ class StatfulClient
       'Content-Type' => 'application/json',
       'M-Api-Token' => @config[:token]
     }
-    response = @http.send_request('PUT', '/tel/v2.0/metrics', message, headers)
 
-    if response.code != '201'
-      @logger.debug("Failed to flush message via http with: #{response.code} - #{response.msg}") unless @logger.nil?
+    @pool.post do
+      begin
+        response = @http.send_request('PUT', '/tel/v2.0/metrics', message, headers)
+
+        if response.code != '201'
+          @logger.debug("Failed to flush message via http with: #{response.code} - #{response.msg}") unless @logger.nil?
+        end
+      rescue StandardError => ex
+        @logger.debug("Statful: #{ex} on #{@config[:host]}:#{@config[:port]}") unless @logger.nil?
+        false
+      end
     end
   end
 
@@ -319,4 +333,16 @@ class StatfulClient
       symbolize[self]
     end
   end
+
+  # Custom Queue implementation to add a to_a method
+  #
+  # @private
+  class MyQueue < Queue
+    # Transform Queue to Array
+    #
+    # @return [Array] queue as array
+    def to_a
+      [].tap { |array| array << pop until empty? }
+    end
+  end
 end

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: statful-client
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.1.0
 platform: ruby
 authors:
 - Miguel Fonseca
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-06 00:00:00.000000000 Z
+date: 2016-10-17 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby-ext
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement