online_migrations 0.30.0 → 0.31.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 428fd65e363b915608f720370a2cd91ec14749cd17933ec4e26351bb4745084a
-  data.tar.gz: ae292bba4cd1b557694f78240a7da761c8d890aa691cc6b485ddd9f71fd339c5
+  metadata.gz: '0872638ad98f95ef8fa347b719f14d548fec58402dd8fc6a06702c9d884f9346'
+  data.tar.gz: 572c037d2bf35cf048b8e1ee45fb0e464d375a3adec18264ba3e8f78b5cd433b
 SHA512:
-  metadata.gz: 59c0573c24d75488c849fa746d6346c31e439d66032725264ce724e97ab8ffbd16cebb03a57ad75071e2b29bf884be0b47696abdcadc72d974effaafd73fe8b4
-  data.tar.gz: 73de95254d001d4d285a48ad4d3adb60726ab6bf0bb86b085dde8466b6e1e85fbb668738f09121c38718504ad253615afb3da5e24f56bc48bd01ce9d369cdd4c
+  metadata.gz: 8da4b37bc6b6e88803a478844b32aa6d04d28b95ab6a147caea69a22e1ce512a70e0df9e6196ceb0cbc87a5c1a164f01d927adcb054d36d2aa98e56ecac854c9
+  data.tar.gz: f0a3f79d9ba42b596f27861ed9b046d82fd3805b47d62c7745b4bac7e69033904a6bbd90ea9d137370679e6e4ea8a42672cd31378f28b8a6b5d52141837c4d86

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 ## master (unreleased)
 
+## 0.31.0 (2025-10-22)
+
+- Store backtraces for failed background data migrations jobs
+- Make `add_column` idempotent
+- Fix `NullLockRetrier#with_lock_retries` method signature
+
 ## 0.30.0 (2025-10-17)
 
 - Fix `remove_check_constraint` when using `:if_exists`

data/lib/online_migrations/background_data_migrations/migration_job.rb

@@ -6,6 +6,8 @@ module OnlineMigrations
     class MigrationJob
       include Sidekiq::IterableJob
 
+      sidekiq_options backtrace: true
+
       sidekiq_retry_in do |count, _exception, jobhash|
         migration_id = jobhash["args"].fetch(0)
         migration = Migration.find(migration_id)

data/lib/online_migrations/lock_retrier.rb

@@ -251,7 +251,7 @@ module OnlineMigrations
     def delay(_attempt, _command = nil, _arguments = [])
     end
 
-    def with_lock_retries(_connection)
+    def with_lock_retries(_connection, _command = nil, *_arguments)
       yield
     end
   end

data/lib/online_migrations/schema_statements.rb

@@ -630,6 +630,19 @@ module OnlineMigrations
       remove_check_constraint(table_name, name: name)
     end
 
+    # Extends default method to be idempotent and automatically recreate invalid indexes.
+    #
+    # @see https://api.rubyonrails.org/v8.1/classes/ActiveRecord/ConnectionAdapters/SchemaStatements.html#method-i-add_column
+    #
+    def add_column(table_name, column_name, type, **options)
+      if column_exists?(table_name, column_name, type, **options)
+        Utils.say("Column was not added because it already exists (this may be due to an aborted migration " \
+                  "or similar) table_name: #{table_name}, column_name: #{column_name}")
+      else
+        super
+      end
+    end
+
     # Adds a reference to the table with minimal locking
     #
     # Active Record adds an index non-`CONCURRENTLY` to references by default, which blocks writes.

data/lib/online_migrations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OnlineMigrations
-  VERSION = "0.30.0"
+  VERSION = "0.31.0"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: online_migrations
 version: !ruby/object:Gem::Version
-  version: 0.30.0
+  version: 0.31.0
 platform: ruby
 authors:
 - fatkodima
 bindir: bin
 cert_chain: []
-date: 2025-10-17 00:00:00.000000000 Z
+date: 2025-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord