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

Xuyang Jia notifications at github.com
Tue Sep 18 02:06:38 UTC 2018


> It's pretty much the same performance optimization I described in #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 introduces quite a bit of duplication, i.e. @gravitystorm probably needs to to through it in much more detail.
> 
> The general concept of bundling many small sql statments into larger ones works regardless of which language issues the sql requests. Still there's some more overhead in Rails processing vs. C++ (about factor 5 in one test case). In any case, we will see at least a 30x speedup.

Yes, I have seen your analysis and effort on cgimap. But it seems the new feature bulk_upload not finished? I just can't wait...

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


More information about the rails-dev mailing list