emque-consuming 1.5.0 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 18f37a4c2c4ca2951d9418cc53d3c4cb3b01ffcb
-  data.tar.gz: cb68592cb698f450c0059caec6ec8b9ccd3b8672
+  metadata.gz: '069aa5e7b83fb25837163e04e478c2b611bbd366'
+  data.tar.gz: cdeb70eb71c537dfce8a1f2df16482a718987415
 SHA512:
-  metadata.gz: b98cc1163c2ea82dc1b73b5e9d05e1851e9baeb7c65ea5c9e84a2f01b7199d3bd80893b22760aeb236d71b71e19df88b53df1bb3e7cce7f615815a74c0b2bc05
-  data.tar.gz: a49054f4f05989287538ea0c1ec25df927e7018641665a9c49c16891106f2fe84113d1723cfeb4dc6493c5fbe17a346db1867114f5d921779a7c46b4c0d86c3d
+  metadata.gz: eb34ba19dba915d3826c291a739d7f6cfab1f58dc1ea5a890180dcccb7744ba0a78d6d4493e152a5d0080124ae42d5034f3e49567bcb01079b1ead5bdee4fdcf
+  data.tar.gz: f0695229f7c89bc76950e968d3992c8c77c51888c44a11e9eeae178a0974769913086a3bab6c5dbc7787486bbcceaed28e123cd1a3efaa26c106a9de2107abd2

data/CHANGELOG.md

@@ -1,5 +1,6 @@
 # Emque Consuming CHANGELOG
 
+- [Re-enable pipe-ruby `raise_on_error` option to fix automatic retries directing messages to the error queue](https://github.com/emque/emque-consuming/pull/75) 1.6.0
 - [Update the puma gem to allow v3](https://github.com/emque/emque-consuming/pull/72) 1.5.0
 - [Disable pipe-ruby `raise_on_error` option to prevent duplicate erorrs](https://github.com/emque/emque-consuming/pull/74) 1.4.0
 - [Update minimum Ruby version to 2.3](https://github.com/emque/emque-consuming/pull/68) 1.3.0

data/lib/emque/consuming/adapters/rabbit_mq/delayed_message_worker.rb

@@ -69,8 +69,6 @@ module Emque
               ::Emque::Consuming::Consumer.new.consume(:process, message)
               channel.ack(delivery_info.delivery_tag)
             rescue StandardError => exception
-              logger.error "#{log_prefix} #{exception.class}: #{exception.message}"
-              exception.backtrace.each { |bt| logger.error "#{log_prefix} #{bt}" }
               if retryable_errors.any? { |error| exception.class.to_s =~ /#{error}/ }
                 retry_error(delivery_info, metadata, payload, exception)
               else

data/lib/emque/consuming/adapters/rabbit_mq/error_worker.rb

@@ -15,7 +15,7 @@ module Emque
           def retry_errors
             logger.info "#{log_prefix} starting"
             channel.open if channel.closed?
-            error_queue.message_count.times do
+            [error_queue.message_count, 100].min.times do
               delivery_info, properties, payload = error_queue.pop(
                 {:manual_ack => true}
               )
@@ -56,8 +56,6 @@ module Emque
               ::Emque::Consuming::Consumer.new.consume(:process, message)
               channel.ack(delivery_info.delivery_tag)
             rescue StandardError => exception
-              logger.error "#{log_prefix} #{exception.class}: #{exception.message}"
-              exception.backtrace.each { |bt| logger.error "#{log_prefix} #{bt}" }
               channel.nack(delivery_info.delivery_tag)
             end
           end

data/lib/emque/consuming/adapters/rabbit_mq/worker.rb

@@ -79,8 +79,6 @@ module Emque
               ::Emque::Consuming::Consumer.new.consume(:process, message)
               channel.ack(delivery_info.delivery_tag)
             rescue StandardError => exception
-              logger.error "#{log_prefix} #{exception.class}: #{exception.message}"
-              exception.backtrace.each { |bt| logger.error "#{log_prefix} #{bt}" }
               if enable_delayed_message
                 begin
                   publish_to_delayed_message(delivery_info, metadata, payload)

data/lib/emque/consuming/consumer/common.rb

@@ -27,7 +27,7 @@ module Emque
         def pipe_config
           @pipe_config ||= Pipe::Config.new(
             :error_handlers => [method(:handle_error)],
-            :raise_on_error => false,
+            :raise_on_error => true,
             :stop_on => ->(msg, _, _) { !(msg.respond_to?(:continue?) && msg.continue?) }
           )
         end

data/lib/emque/consuming/version.rb

@@ -1,5 +1,5 @@
 module Emque
   module Consuming
-    VERSION = "1.5.0"
+    VERSION = "1.6.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: emque-consuming
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.6.0
 platform: ruby
 authors:
 - Ryan Williams
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-04 00:00:00.000000000 Z
+date: 2019-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: celluloid