aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTeddy Wing2018-01-09 15:06:47 +0100
committerTeddy Wing2018-01-09 15:06:47 +0100
commit7c9e777704ee21f89630baa61e79c5baecc187bd (patch)
tree54fcf8ab5a5d95d72d8183efbed421ea99bcc0cf
parentb39cbc049bfa391e9d9fb71852bd8ce34e1d34a4 (diff)
downloadchouette-core-7c9e777704ee21f89630baa61e79c5baecc187bd.tar.bz2
MailerJob: Add comment describing retry behaviour
Now that all other workers don't retry automatically (see 3487ce6f34ef187a2a8daf170440f51f659c46b7), add a comment here to clarify that this worker is different, and we explicitly want it to retry. The disabling of retries only affects Sidekiq jobs. Refs #5495
-rw-r--r--app/jobs/mailer_job.rb3
1 files changed, 3 insertions, 0 deletions
diff --git a/app/jobs/mailer_job.rb b/app/jobs/mailer_job.rb
index 3918745b8..eb3250a27 100644
--- a/app/jobs/mailer_job.rb
+++ b/app/jobs/mailer_job.rb
@@ -1,6 +1,9 @@
class MailerJob < ActiveJob::Base
# No need to specify queue it's already used mailers queue
+ # This job will be retried, unlike Sidekiq jobs which are configured
+ # to not retry
+
def perform klass, action, params
klass.constantize.public_send(action, *params).deliver_later
end