online_migrations 0.19.5 → 0.19.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9cb60167b94f5ce8556b3f9835949c86f9be4c523eb2c35494de578dea31a2bf
-  data.tar.gz: 20d9d6fcf509bbc25987a2e3bce165c927cacd03502aa908c02aba37dd284774
+  metadata.gz: cd8711d56259535675633074868019a5c4b3de068bb30d783c9ae16dc906afb1
+  data.tar.gz: 3b417e176d143092d69e4551c1f96a32e1a2acf1d74d268fc33f9b5a598be553
 SHA512:
-  metadata.gz: 03f82fe701b6d2145963f1f2220ac9ed127dc826885455134afc3bb547197d21a1c0307156b6d8b5402c0a521332328eebf45d96a3bb2183804055d866cd3af2
-  data.tar.gz: 94f0bfd73299897757a8b836a9cbd995bb9313c971a6e0076079b783955c26da239098b84df391096128fc0e060ba66ca69e61e16a84013c12d613c7fe4064ce
+  metadata.gz: ad26fd5662f2636479b697933042b8d7e60b4ee42f136ddeb8cff8256c65a4491b8017d09d2da849faf65cb6f8d4ecc95595816b7669c8364e9f31bbe75232c4
+  data.tar.gz: 21fec027462ff6f96baaf0fc65d186fc68fefbb50fd3de6af8f561aae691e6f9909d70d7053557d39f0a56ea498841816c2853cce0b1acef161411b2d968b196

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## master (unreleased)
 
+## 0.19.6 (2024-09-26)
+
+- Fix `add_refernce_concurrently` when adding non polymorphic references
+- Fix progress for background migrations with small number of records
+
 ## 0.19.5 (2024-09-20)
 
 - Fix `add_reference_concurrently` when adding polymorphic references

data/lib/online_migrations/background_migrations/migration.rb

@@ -138,7 +138,7 @@ module OnlineMigrations
             progresses.sum.round(2)
           end
         elsif rows_count
-          if rows_count > 0
+          if rows_count > 0 && rows_count > batch_size
             jobs_rows_count = migration_jobs.succeeded.sum(:batch_size)
             # The last migration job may need to process the amount of rows
             # less than the batch size, so we can get a value > 1.0.
@@ -183,14 +183,14 @@ module OnlineMigrations
       def retry
         if composite? && failed?
           children.failed.each(&:retry)
-          running!
+          enqueued!
           true
         elsif failed?
           iterator = BatchIterator.new(migration_jobs.failed)
           iterator.each_batch(of: 100) do |batch|
             batch.each(&:retry)
           end
-          running!
+          enqueued!
           true
         else
           false

data/lib/online_migrations/background_migrations/migration_job.rb

@@ -81,7 +81,7 @@ module OnlineMigrations
               error_message: nil,
               backtrace: nil
             )
-            migration.running! if migration.failed?
+            migration.enqueued! if migration.failed?
           end
           true
         else

data/lib/online_migrations/schema_statements.rb

@@ -655,7 +655,7 @@ module OnlineMigrations
         add_column(table_name, column_name, type, null: allow_null)
       end
 
-      if !column_exists?(table_name, type_column_name)
+      if options[:polymorphic] && !column_exists?(table_name, type_column_name)
         allow_null = options[:polymorphic].is_a?(Hash) ? options[:polymorphic][:null] : true
         add_column(table_name, type_column_name, :string, null: allow_null)
       end

data/lib/online_migrations/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: online_migrations
 version: !ruby/object:Gem::Version
-  version: 0.19.5
+  version: 0.19.6
 platform: ruby
 authors:
 - fatkodima
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-19 00:00:00.000000000 Z
+date: 2024-09-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord