deimos-kafka 1.0.0.pre.beta17 → 1.0.0.pre.beta18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30b4706911af70b7e4662afdecf5b547583b74dbdebce492a7e57a9b1c9d504e
-  data.tar.gz: d5583aefa2e976497d8190398f0b04b2d07e33f6ff8f5ac79089bffa850b53f9
+  metadata.gz: aa0b3386a91de8df444dc0dd95daa79965dd2458d7f76088ce9f3aab9812838e
+  data.tar.gz: 59ac90f569462a2493e3ecf7920a1114ff275db52bda1f2ac0f4777828cc3b97
 SHA512:
-  metadata.gz: 329ad07cc57cc869c1f9a843d74f70a428c7706882ce5bcabe8006afbc0e557f5931649ea4871973d21b5edd8d5e892dcf82ae5e0231a21e8b6e8021c2da187c
-  data.tar.gz: dc46360e6061b586b127888ac784b946fde6e9e5efe73173b8963aa1345fd65b58fe174e4fc69db37d9093ef17549c436087e1d35a73df555135c1d430f19a3b
+  metadata.gz: 1c5ec5ccba522c2e668cb71e4d46254eacf8af7c190918567e62b70924cf1d382f5a03ce75df9ea2f8f5e5d55a66f34d259d702419f3620891d1c1d421c00531
+  data.tar.gz: 39644011b97084a32ec248ad7fc39c068c10b0d199744e6a8d2a47f82606093c05a242388a32a90464078abcdecb08df0e186061266a00d6589ddf2eb3cf737e

data/CHANGELOG.md

@@ -5,6 +5,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.0-beta18] - 2019-08-02
+- Fixed crash when sending metrics in a couple of places.
+
 ## [1.0.0-beta17] - 2019-07-31
 - Added `rails deimos:db_producer` rake task.
 - Fixed the DB producer so it runs inline instead of on a separate thread.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    deimos-kafka (1.0.0.pre.beta17)
+    deimos-kafka (1.0.0.pre.beta18)
       avro-patches (~> 0.3)
       avro_turf (~> 0.8)
       phobos (~> 1.8)
@@ -29,8 +29,8 @@ GEM
       multi_json
     avro-patches (0.4.1)
       avro (= 1.8.2)
-    avro_turf (0.8.1)
-      avro (>= 1.7.7, < 1.9)
+    avro_turf (0.9.0)
+      avro (>= 1.7.7, < 1.10)
       excon (~> 0.45)
     coderay (1.1.2)
     concurrent-ruby (1.1.3)
@@ -41,7 +41,7 @@ GEM
     diff-lcs (1.3)
     digest-crc (0.4.1)
     dogstatsd-ruby (4.2.0)
-    excon (0.64.0)
+    excon (0.65.0)
     exponential-backoff (0.0.2)
     ffi (1.11.1)
     formatador (0.2.5)
@@ -127,7 +127,7 @@ GEM
       unicode-display_width (>= 1.4.0, < 1.7)
     rubocop-rspec (1.32.0)
       rubocop (>= 0.60.0)
-    ruby-kafka (0.7.8)
+    ruby-kafka (0.7.9)
       digest-crc
     ruby-progressbar (1.10.1)
     ruby_dep (1.5.0)

data/lib/deimos.rb

@@ -78,12 +78,12 @@ module Deimos
     # Start the DB producers to send Kafka messages.
     # @param thread_count [Integer] the number of threads to start.
     def start_db_backend!(thread_count: 1)
-      if self.config.publish_backend != :db
-        raise("Publish backend is not set to :db, exiting")
+      if self.config.publish_backend != :db # rubocop:disable Style/IfUnlessModifier
+        raise('Publish backend is not set to :db, exiting')
       end
 
       if thread_count.nil? || thread_count.zero?
-        raise("Thread count is not given or set to zero, exiting")
+        raise('Thread count is not given or set to zero, exiting')
       end
 
       producers = (1..thread_count).map do

data/lib/deimos/instrumentation.rb

@@ -51,8 +51,8 @@ module Deimos
         producer = batch.first.metadata[:producer_name]
         payloads = batch.map { |m| m.metadata[:decoded_payload] }
 
-        Deimos.metrics&.count('publish_error', payloads.size,
-                              tags: %W(topic:#{topic}))
+        Deimos.config.metrics&.count('publish_error', payloads.size,
+                                     tags: %W(topic:#{topic}))
         Deimos.instrument(
           'produce_error',
           producer: producer,

data/lib/deimos/utils/db_producer.rb

@@ -83,7 +83,7 @@ module Deimos
       def produce_messages(batch)
         @logger.debug("Publishing #{batch.size} messages to #{@current_topic}")
         producer.publish_list(batch)
-        Deimos.metrics&.increment(
+        Deimos.config.metrics&.increment(
           'publish',
           tags: %W(status:success topic:#{@current_topic}),
           by: batch.size

data/lib/deimos/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Deimos
-  VERSION = '1.0.0-beta17'
+  VERSION = '1.0.0-beta18'
 end

data/lib/generators/deimos/db_backend/templates/migration

@@ -10,7 +10,7 @@ class <%= migration_class_name %> < ActiveRecord::Migration<%= migration_version
 
     add_index :kafka_messages, [:topic, :id]
 
-    create_table :kafka_topic_info, force: true do |t|
+    create_table :kafka_topic_info, force: true do |t| # rubocop:disable Rails/CreateTableWithTimestamps
       t.string :topic, null: false
       t.string :locked_by
       t.datetime :locked_at

data/lib/generators/deimos/db_backend/templates/rails3_migration

@@ -10,7 +10,7 @@ class <%= migration_class_name %> < ActiveRecord::Migration<%= migration_version
 
     add_index :kafka_messages, [:topic, :id]
 
-    create_table :kafka_topic_info, force: true do |t|
+    create_table :kafka_topic_info, force: true do |t| # rubocop:disable Rails/CreateTableWithTimestamps
       t.string :topic, null: false
       t.string :locked_by
       t.datetime :locked_at

data/lib/tasks/deimos.rake

@@ -19,7 +19,7 @@ namespace :deimos do
   task db_producer: :environment do
     ENV['DEIMOS_RAKE_TASK'] = 'true'
     STDOUT.sync = true
-    Rails.logger.info("Running deimos:db_producer rake task.")
+    Rails.logger.info('Running deimos:db_producer rake task.')
     thread_count = ENV['THREADS'].presence || 1
     Deimos.start_db_backend!(thread_count: thread_count)
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deimos-kafka
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.beta17
+  version: 1.0.0.pre.beta18
 platform: ruby
 authors:
 - Daniel Orner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-31 00:00:00.000000000 Z
+date: 2019-08-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: avro-patches