[openstreetmap/openstreetmap-website] Migrate some sequences to use bigints (PR #4391)

Andy Allan notifications at github.com
Wed Dec 6 14:55:39 UTC 2023


These primary keys were converted to bigints in migrations, but the sequences were left unmentioned. If the original migrations are run on postgresql 10.0+, then this leads to a mismatch in column types vs sequence types. This migration fixes these mismatches.

If the original migrations were run on postgresql < 10, all sequences were bigints anyway, and this migration is a no-op.

If the sequence is a bigint, then postgresql doesn't output that fact in the statement dump.

Refs #4298
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/4391

-- Commit Summary --

  * Migrate some sequences to use bigints

-- File Changes --

    A db/migrate/20231206141457_alter_sequences_bigint.rb (17)
    M db/structure.sql (4)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/4391.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4391.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/4391 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231206/73f20880/attachment.htm>


More information about the rails-dev mailing list