[openstreetmap/openstreetmap-website] Optional Trace import using ActiveJob (#2120)

Andy Allan notifications at github.com
Mon Jan 28 18:42:15 UTC 2019


> Bulk INSERT is quite crucial from a performance pov, otherwise this thing is barely usable

I worked on it last week too, after I made the pull request, and it was [pretty straightforward](https://github.com/gravitystorm/openstreetmap-website/commit/74e1b98ea2701b722ebbb124e03f272848c51676). I noticed that the way the import method currently works involved multiple passes at the XML parsing, so I want to look at that too. But in general, I'm a fan of incremental pull requests, to make the reviews more straightforward.

I'll work on this further next week.

-- 
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#issuecomment-458251497
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190128/29501594/attachment.html>


More information about the rails-dev mailing list