[openstreetmap/openstreetmap-website] Lock changesets during a diff upload (PR #6379)

Pablo Brasero notifications at github.com
Mon Sep 29 10:50:45 UTC 2025


pablobm left a comment (openstreetmap/openstreetmap-website#6379)

Thank you, that makes sense :+1: 

One question: does the `FOR UPDATE` need to be in a transaction? Reading the Postgres docs I'm understanding yes, and that's why the `.lock` is put within `Changeset.transaction`. But if this wasn't within a transaction, what would be the behaviour? I see that `FOR UPDATE` still happens, but I wonder how log the lock lasts for in that case.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6379#issuecomment-3346284890
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6379/c3346284890 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250929/7a3777d3/attachment.htm>


More information about the rails-dev mailing list