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/c2379765...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to