### Description PR adds migration script for copying notes descriptions, user_ids and user_ips from notes first comments to notes table records. In 2 commits we propose 2 variants of migration script (not sure which is more appropriate for the situation because of large number of notes):
1. basic version - for every `batch of notes`, dynamically generates SQL query for copying values and executes it (simpler to understand, ..). 2. enhanced version - same as previous but keeps track about notes for which migration failed (for some reason?) and retries again later (more robust, ..) ### How has this been tested? Automated and manual tests before and after data migrations (both basic and enhanced one) You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5667 -- Commit Summary -- * Copies description, user_id and user_ip to notes * Enhance backfill migration script with logging, retries, and conversion fixes -- File Changes -- A db/migrate/20250212160355_backfill_note_descriptions.rb (169) M db/structure.sql (1) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/5667.patch https://github.com/openstreetmap/openstreetmap-website/pull/5667.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5667 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev
