[openstreetmap/openstreetmap-website] Trace import: remove activerecord-import gem (PR #5038)
mmd
notifications at github.com
Fri Sep 27 17:40:41 UTC 2024
A few years back, I've introduced both the activerecord-import gem and `import!` method call to speed up Tracepoint imports. Alternative options weren't available then. Now with the recent Rails version, it's a good time to move to the core Rails code.
At least today, we have exactly one location where `insert_all` will be used. Nobody else was doing anything with that gem over the past few years. The only other attempt to use activerecord-import was a performance improved changeset upload implementation which never got merged. So it's somewhat unlikely that we will see this issue in many other places in the future.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5038#issuecomment-2379765989
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5038/c2379765989 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240927/6ded0acf/attachment.htm>
More information about the rails-dev
mailing list