fluent-plugin-amplitude 0.0.8 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 87356ba6323e223e3ce8b3b34424a1f942d2ee58
-  data.tar.gz: 5f4552341e234469911d09c064d692d6851da300
+  metadata.gz: a2df4393337ff57bd66d8282d7d27d00e18fd11f
+  data.tar.gz: 0bb12c933cdfed442fdec1c8cde1fe66e06712b6
 SHA512:
-  metadata.gz: 7afeee3c37fe492271e6db7d04caff6d59277c30f65fac72f24b2427b2085390578e59d80a15f0c0e15f1d5cf5f4f49194e6a43a99aaf4ac05d2198bb443d931
-  data.tar.gz: 056333c1581a72a259ff89e42c9bc9c3ec77d2e7e227bebef478eafd2a6ecc388d47b5d60beed25a876c97c3bd769d92d425f22c52b0a2cd6dba0f46a56b0fd2
+  metadata.gz: 9f3a3f198523457df7d57b82ee7eb6bb75536be0665a8c4ef9e027118878fa09459988092d78913377eaae44251e29d4b70d92cd79a665172dda71e8c57b7dec
+  data.tar.gz: 08ac45b1b9eea96014545ff9c9e9c11b05f524c7de17541125410ca3e323fcc36049df41bdf17b163a9a9641bbf89122d917cdcc466ba50a81a6d486cf8a1eea

data/fluent-plugin-amplitude.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = 'fluent-plugin-amplitude'
-  spec.version       = '0.0.8'
+  spec.version       = '0.0.9'
   spec.authors       = ['Vijay Ramesh']
   spec.email         = ['vijay@change.org']
   spec.summary       = 'Fluentd plugin to output event data to Amplitude'

data/lib/fluent/plugin/out_amplitude.rb

@@ -57,7 +57,9 @@ module Fluent
         if verify_user_and_device(record)
           records << AmplitudeAPI::Event.new(record)
         else
-          log.info("Error: either user_id or device_id must be set for tag #{tag}")
+          log.info(
+            "Error: either user_id or device_id must be set for tag #{tag}"
+          )
         end
       end
 
@@ -93,7 +95,7 @@ module Fluent
     def verify_user_and_device(amplitude_hash)
       user_id = amplitude_hash[:user_id]
       device_id = amplitude_hash[:device_id]
-      return present?(user_id) || present?(device_id)
+      present?(user_id) || present?(device_id)
     end
 
     def extract_user_properties!(amplitude_hash, record)
@@ -123,14 +125,19 @@ module Fluent
 
     def send_to_amplitude(records)
       log.info("sending #{records.length} to amplitude")
-      begin
-        res = AmplitudeAPI.track(records)
+      errors = []
+      until records.empty?
+        res = AmplitudeAPI.track(records.pop(500))
         unless res.response_code == 200
-          raise "Got #{res.response_code} #{res.body} from AmplitudeAPI"
+          errors << [[res.response_code, res.body]]
         end
-      rescue StandardError => e
-        raise AmplitudeError, "Error: #{e.message}"
       end
+      return if errors.empty?
+
+      errors_string = errors.map do |code, body|
+        "Response: #{code} Body: #{body}"
+      end
+      raise AmplitudeError, "Errors: #{errors_string}"
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-amplitude
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors:
 - Vijay Ramesh
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-17 00:00:00.000000000 Z
+date: 2016-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd