dispatch-rider 1.8.4 → 1.8.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: df7b869b4f2f8bf0c8a1127c02e9431f133af311
-  data.tar.gz: acf144ab31fb5752f18d765591fbfd46a2b028ff
+  metadata.gz: 71dd07d76eb7f93700d28c2127d426803737880c
+  data.tar.gz: 1b5a4ae16a98078e237524a5457e0ac582ea4a6e
 SHA512:
-  metadata.gz: 06e7944d5a52c584460f5616f45a23e2a137bb16b505c653e3f301d611fafcc3f4e7e8d9f1eac90f473f4cd085da077c1b299422e7f249a32ed193d7891afe8c
-  data.tar.gz: 64e66b6b04caee2d4c090d5f3a120315efe772ddff90938c67175d28d2ff6becd770ffe570e42ea5695d6c3cdebed47fc8ecd7960cdd43e52dbe1622f0657bbc
+  metadata.gz: b3f941f817e835ec7b4b30fd8dcbbf7aac35edab8d569606042525c07c1b46056c81f38dec351802bd0b162ff614a1df66333cff4a034f5064ba352084472ecf
+  data.tar.gz: 72aed806508c13450029709b5691c26a1cec984e4da16982de6980146b0023ef2f556e235af2d7a84f8698dbc3dac06338aa27bee9c6377f8539fc5fc291d68f

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Change Log
 
+## [v1.8.4](https://github.com/payrollhero/dispatch-rider/tree/v1.8.4) (2016-02-09)
+
+[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v1.8.3...v1.8.4)
+
+**Merged pull requests:**
+
+- Fix appsignal integration [\#73](https://github.com/payrollhero/dispatch-rider/pull/73) ([vincentpaca](https://github.com/vincentpaca))
+
+- Update appsignal integration [\#72](https://github.com/payrollhero/dispatch-rider/pull/72) ([vincentpaca](https://github.com/vincentpaca))
+
+## [v1.8.3](https://github.com/payrollhero/dispatch-rider/tree/v1.8.3) (2016-01-19)
+
+[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v1.8.2...v1.8.3)
+
 ## [v1.8.2](https://github.com/payrollhero/dispatch-rider/tree/v1.8.2) (2016-01-19)
 
 [Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v1.8.1...v1.8.2)
@@ -348,6 +362,8 @@
 
 - Added process naming to dispatcher. [\#23](https://github.com/payrollhero/dispatch-rider/pull/23) ([link664](https://github.com/link664))
 
+- Fail job after running error handler. [\#27](https://github.com/payrollhero/dispatch-rider/pull/27) ([link664](https://github.com/link664))
+
 ## [v0.0.5](https://github.com/payrollhero/dispatch-rider/tree/v0.0.5) (2013-08-20)
 
 [Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/2013-06-06-13-08-PST...v0.0.5)
@@ -394,6 +410,10 @@
 
 - Fix double serialize on file system bug [\#15](https://github.com/payrollhero/dispatch-rider/pull/15) ([more-ron](https://github.com/more-ron))
 
+- Add publisher configure method [\#16](https://github.com/payrollhero/dispatch-rider/pull/16) ([more-ron](https://github.com/more-ron))
+
+-  Initial implementation of hash based configuration to satisfy issue \#10 [\#11](https://github.com/payrollhero/dispatch-rider/pull/11) ([jimbomt](https://github.com/jimbomt))
+
 ## [2013-05-28-15-13+08-00](https://github.com/payrollhero/dispatch-rider/tree/2013-05-28-15-13+08-00) (2013-05-28)
 
 [Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/2013-05-28...2013-05-28-15-13+08-00)
@@ -408,7 +428,7 @@
 
 ## [2013-05-28](https://github.com/payrollhero/dispatch-rider/tree/2013-05-28) (2013-05-28)
 
-[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.3...2013-05-28)
+[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.2...2013-05-28)
 
 **Merged pull requests:**
 
@@ -416,13 +436,13 @@
 
 - Another stab at the initial implementation for issue \#10 [\#12](https://github.com/payrollhero/dispatch-rider/pull/12) ([piotrb](https://github.com/piotrb))
 
-## [v0.0.3](https://github.com/payrollhero/dispatch-rider/tree/v0.0.3) (2013-05-22)
+## [v0.0.2](https://github.com/payrollhero/dispatch-rider/tree/v0.0.2) (2013-05-22)
 
-[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.2...v0.0.3)
+[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.3...v0.0.2)
 
-## [v0.0.2](https://github.com/payrollhero/dispatch-rider/tree/v0.0.2) (2013-05-22)
+## [v0.0.3](https://github.com/payrollhero/dispatch-rider/tree/v0.0.3) (2013-05-22)
 
-[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.1...v0.0.2)
+[Full Changelog](https://github.com/payrollhero/dispatch-rider/compare/v0.0.1...v0.0.3)
 
 **Merged pull requests:**
 

data/lib/dispatch-rider/queue_services/aws_sqs.rb

@@ -24,20 +24,22 @@ module DispatchRider
         end
       end
 
-      def pop(&block)
-        begin
-          queue.receive_message do |raw_item|
-            obj = SqsReceivedMessage.new(construct_message_from(raw_item), raw_item, queue)
+      def pop
+        raw_item = queue.receive_message
+        if raw_item.present?
+          obj = SqsReceivedMessage.new(construct_message_from(raw_item), raw_item, queue)
 
-            visibility_timeout_shield(obj) do
-              raise AbortExecution, "false received from handler" unless block.call(obj)
-              obj
-            end
+          visibility_timeout_shield(obj) do
+            raise AbortExecution, "false received from handler" unless yield(obj)
+            obj
+          end
 
+          with_retries(max_tries: 3) do
+            raw_item.delete
           end
-        rescue AbortExecution
-          # ignore, it was already handled, just need to break out if pop
         end
+      rescue AbortExecution
+        # ignore, it was already handled, just need to break out if pop
       end
 
       def insert(item)

data/lib/dispatch-rider/version.rb

@@ -1,4 +1,4 @@
 # This file specifies the current version of the gem.
 module DispatchRider
-  VERSION = "1.8.4"
+  VERSION = "1.8.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dispatch-rider
 version: !ruby/object:Gem::Version
-  version: 1.8.4
+  version: 1.8.5
 platform: ruby
 authors:
 - Suman Mukherjee
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-09 00:00:00.000000000 Z
+date: 2016-02-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport