[openstreetmap/openstreetmap-website] Optional Trace import using ActiveJob (#2120)
Andy Allan
notifications at github.com
Thu Jan 24 20:47:21 UTC 2019
gravitystorm commented on this pull request.
> @@ -0,0 +1,10 @@
+class TraceDestroyerJob < ApplicationJob
+ queue_as :default
+
+ def perform(trace)
+ trace.destroy
+ rescue StandardError => ex
+ logger.info ex.to_s
+ ex.backtrace.each { |l| logger.info l }
Just copy+pasted from the previous daemon code. I agree, it makes sense to remove the exception handling from here.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2120#discussion_r250770559
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190124/47a48e29/attachment.html>
More information about the rails-dev
mailing list