uber-archive / cherami-server

Distributed, scalable, durable, and highly available message queue system. This project is deprecated and not maintained.
https://eng.uber.com/cherami/
MIT License
1.42k stars 102 forks source link

Fix retryableEventHandler logic #324

Closed thuningxu closed 6 years ago

thuningxu commented 6 years ago

If event.Handle() returns non-retryable error, the original logic will immediately retry because the case is not handled.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.02%) to 66.65% when pulling 706844fc0cdcf0dcc65b6e6d32fa10679b239eac on retry into 65a1bb1404b63ce07025a2a3f6d34187fddce030 on master.