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

mmd notifications at github.com
Mon Jan 28 12:20:48 UTC 2019


> There's obviously two major limitations with this code right now (the lack of the animated gif for the large picture, and the 1-sql-insert-per-tracepoint thing), but I'd rather see those fixed those in future PRs rather than doing everything in one PR.

Animated gif support may be a bit tricky, as the repo owner hasn't been all that responsive lately. Maybe we'd have to fork it, and run our version of it, and find someone to finialize that animated gif part? 

Bulk INSERT is quite crucial from a performance pov, otherwise this thing is barely usable. Given that we already have that code lying around (https://github.com/openstreetmap/openstreetmap-website/issues/1852#issuecomment-427116280), and @tomhughes already reviewed a few iterations, I don't see why we couldn't integrate this part right away.

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


More information about the rails-dev mailing list