[openstreetmap/openstreetmap-website] add bulk_upload (#1995)

mmd notifications at github.com
Mon Sep 17 21:05:55 UTC 2018


It's pretty much the same performance optimization I described in https://github.com/openstreetmap/openstreetmap-website/issues/1710 and also implemented in cgimap. I tested a few easy cases via url rewriting (upload --> bulk_upload), and they seemed ok. The current code seems to be duplicating quite a lot of stuff, i.e. @gravitystorm probably wants to to through the code.

The general concept of bundling many small sql statments into larger ones works, regardless of which language issues the requests. Still there's some more overhead in Rails processing (about factor 5 in one test case).

-- 
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/1995#issuecomment-422171257
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180917/252503b7/attachment.html>


More information about the rails-dev mailing list