[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