online_migrations 0.27.1 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8a3658cc979ead6b8dea71140f3ba95b0f7a4a4c26a27891f49a5b6925c94e59
-  data.tar.gz: 7d3ab5e09422e28078543ac67fa0ecae1a7afe9d8e29f67e55d6b772fb708e50
+  metadata.gz: bcbf178b59ac7d856b8b3b44640aa30bd5bb0bc61c334d90925562da1ca23782
+  data.tar.gz: 3034174c45f26e4e70a338faa8074bcb4c926160202f6dd85dc43d6ffc5b7101
 SHA512:
-  metadata.gz: 43ae3f3ebffa5491d4ac9b99b987b048702b9142e9a4ae49182b705b9c81a562acbea6dc1580ef76e49a9124adf2d662723451b6d03003f10f05c1d39ff4a33e
-  data.tar.gz: 5793df9ffd71454f2edb251d079439c5dfb0cfe212a30360654cbb5622c23fcd47a63e2a8cdd6b8d4df2bd7d3b39ec4d29eede4a99519264075e65956bdd1b55
+  metadata.gz: 228cbf5deacedaf2017b86a7c6f005c89d454be0fdfe05358d1c33414fa1349cac46fa602901bdb9cf16f2b9bc8f73b6f7e8a935ff754a623182083032d649a4
+  data.tar.gz: 3b05af5c90a084eb64ef0f711e0d9ecb88a0e28d73ae3aa3720bbea525454a63061525aa56c56d251fc8044c6741f9cc94258f6eaf2b1cbd4325c01be86390e7

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## master (unreleased)
 
+## 0.28.0 (2025-06-26)
+
+- Support renaming columns and tables for tables within custom schemas
+- Include operation name into the background schema migrations names
+
 ## 0.27.1 (2025-05-08)
 
 - Fix background data migrations to enumerate using the correct shard

data/lib/generators/online_migrations/templates/initializer.rb.tt

@@ -110,6 +110,9 @@ OnlineMigrations.configure do |config|
   # The number of seconds that must pass before the cancelling or pausing data migration is considered stuck.
   config.background_data_migrations.stuck_timeout = 5.minutes
 
+  # The pause interval between each data migration's `process` method execution (in seconds).
+  config.background_data_migrations.iteration_pause = Rails.env.production? ? 0.02 : 0
+
   # The callback to perform when an error occurs during the data migration.
   # config.background_data_migrations.error_handler = ->(error, errored_migration) do
   #   Bugsnag.notify(error) do |notification|

data/lib/online_migrations/background_schema_migrations/migration_helpers.rb

@@ -28,7 +28,7 @@ module OnlineMigrations
         schema_creation = ActiveRecord::ConnectionAdapters::PostgreSQL::SchemaCreation.new(self)
         definition = schema_creation.accept(create_index)
 
-        enqueue_background_schema_migration(index.name, table_name, definition: definition, **migration_options)
+        enqueue_background_schema_migration("Add index #{index.name}", table_name, definition: definition, **migration_options)
       end
 
       def remove_index_in_background(table_name, column_name = nil, name:, **options)
@@ -42,7 +42,7 @@ module OnlineMigrations
         end
 
         definition = "DROP INDEX CONCURRENTLY IF EXISTS #{quote_column_name(name)}"
-        enqueue_background_schema_migration(name, table_name, definition: definition, **migration_options)
+        enqueue_background_schema_migration("Remove index #{name}", table_name, definition: definition, **migration_options)
       end
 
       def validate_foreign_key_in_background(from_table, to_table = nil, **options)
@@ -62,7 +62,7 @@ module OnlineMigrations
           ALTER TABLE #{quote_table_name(table_name)}
           VALIDATE CONSTRAINT #{quote_table_name(constraint_name)}
         SQL
-        enqueue_background_schema_migration(constraint_name, table_name, definition: definition, **options)
+        enqueue_background_schema_migration("Validate #{constraint_name}", table_name, definition: definition, **options)
       end
 
       # Ensures that the background schema migration with the provided migration name succeeded.
@@ -78,7 +78,7 @@ module OnlineMigrations
       #   ensure_background_schema_migration_succeeded("index_users_on_email")
       #
       def ensure_background_schema_migration_succeeded(migration_name)
-        migrations = Migration.where(migration_name: migration_name).to_a
+        migrations = Migration.where("migration_name ILIKE ?", "%#{migration_name}%").to_a
 
         if migrations.empty?
           Utils.raise_in_prod_or_say_in_dev("Could not find background schema migration(s): '#{migration_name}'.")
@@ -97,12 +97,13 @@ module OnlineMigrations
         if connection_class_name
           klass = connection_class_name.constantize
           connection_class = Utils.find_connection_class(klass)
-          # Normalize to the real connection class name.
-          connection_class_name = connection_class.name
         else
           connection_class = ActiveRecord::Base
         end
 
+        # Normalize to the real connection class name.
+        connection_class_name = connection_class.name
+
         shards = Utils.shard_names(connection_class)
         shards = [nil] if shards.size == 1
 

data/lib/online_migrations/schema_cache.rb

@@ -52,15 +52,14 @@ module OnlineMigrations
     private
       def renamed_table?(connection, table_name)
         table_renames = OnlineMigrations.config.table_renames
-        if table_renames.key?(table_name)
-          views = connection.views
-          table_renames[table_name] if views.include?(table_name)
+        if table_renames.key?(table_name) && connection.view_exists?(table_name)
+          table_renames[table_name]
         end
       end
 
       def renamed_column?(connection, table_name)
         column_renames = OnlineMigrations.config.column_renames
-        column_renames.key?(table_name) && connection.views.include?(table_name)
+        column_renames.key?(table_name) && connection.view_exists?(table_name)
       end
 
       def column_rename_table(table_name)
@@ -131,14 +130,21 @@ module OnlineMigrations
       def renamed_table?(pool, table_name)
         table_renames = OnlineMigrations.config.table_renames
         if table_renames.key?(table_name)
-          views = pool.with_connection(&:views)
-          table_renames[table_name] if views.include?(table_name)
+          view_exists = pool.with_connection do |connection|
+            connection.view_exists?(table_name)
+          end
+
+          table_renames[table_name] if view_exists
         end
       end
 
       def renamed_column?(pool, table_name)
         column_renames = OnlineMigrations.config.column_renames
-        column_renames.key?(table_name) && pool.with_connection(&:views).include?(table_name)
+        return false if !column_renames.key?(table_name)
+
+        pool.with_connection do |connection|
+          connection.view_exists?(table_name)
+        end
       end
 
       def column_rename_table(table_name)

data/lib/online_migrations/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: online_migrations
 version: !ruby/object:Gem::Version
-  version: 0.27.1
+  version: 0.28.0
 platform: ruby
 authors:
 - fatkodima
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-08 00:00:00.000000000 Z
+date: 2025-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord