online_migrations 0.14.0 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 296a1517b032cbd6d5fa3948f60f1a4f20c4142c8f4c7676c43998f9e941925d
-  data.tar.gz: f61a7c6728a6ea1e834be61187b61e3ddf004f4314003752ec3ccf9a5b79afd9
+  metadata.gz: 2b1b826df875fae97622c702205853da01d67153172ef73fbb0edf28a2f7ff60
+  data.tar.gz: 18ae161255543e7ab7c266efc94f1ca166a344aa18b92dd4ab2f37209fb42547
 SHA512:
-  metadata.gz: fd04da6b57f9862726ee45e6ed13b1032632099a089e26e691717b2e93009269b7f8a8cf3c828501ef97a86d3d13731d4cadb77d0b48f0cf25fea172502780c1
-  data.tar.gz: '089a7f40afc28b9671669764ee98c760e4ee8649aab67a4d814b22ca8f5c52cf76afc8a5c9f95244d8be7b879d2b714fa38f07fff3c741bcc91e450077c96801'
+  metadata.gz: 705e4ce816bd8b4fcbcb78871589274bdf37acca01fbdf031883e87403a403782d28990ebed10acdf57adcbf3fb4e30e1dbf187123029bf6b1b7d31f26ccd3ca
+  data.tar.gz: 60ea354440b96645c5c03345d60793a66ba61d8aa1b946737098bc1449ba67e0b24caeb4581c525b3b014c5dd44b0ac28e0c539c8e3454ad67a3a1610cffdd79

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 ## master (unreleased)
 
+## 0.15.0 (2024-03-19)
+
+- Reraise errors when running background migrations inline
+- Add `remove_background_migration` migration helper
+- Allow adding bigint foreign keys referencing integer primary keys
+- Fix `add_reference_concurrently` to check for mismatched key types
+
+## 0.14.1 (2024-02-21)
+
+- Fix `MigrationRunner` to consider `run_background_migrations_inline` proc
+
 ## 0.14.0 (2024-02-01)
 
 - Add ability to configure whether background migrations should be run inline

data/README.md

@@ -191,7 +191,7 @@ end
 
    ```ruby
    class User < ApplicationRecord
-     self.ignored_columns = ["name"]
+     self.ignored_columns += ["name"]
    end
    ```
 
@@ -206,7 +206,7 @@ end
    end
    ```
 
-4. Remove column ignoring from `User` model
+4. Remove column ignoring from step 1
 5. Deploy
 
 ### Adding a column with a default value
@@ -488,7 +488,7 @@ It will use a combination of a VIEW and column aliasing to work with both column
 
    ```ruby
    class User < ApplicationRecord
-     self.ignored_columns = ["name"]
+     self.ignored_columns += ["name"]
    end
    ```
 
@@ -1172,7 +1172,7 @@ A safer approach is to:
 
    ```ruby
    class User < ApplicationRecord
-     self.ignored_columns = ["type"]
+     self.ignored_columns += ["type"]
    end
    ```
 

data/docs/background_migrations.md

@@ -30,7 +30,8 @@ A generator is provided to create background migrations. Generate a new backgrou
 $ bin/rails generate online_migrations:background_migration backfill_project_issues_count
 ```
 
-This creates the background migration file `lib/online_migrations/background_migrations/backfill_project_issues_count.rb`.
+This creates the background migration file `lib/online_migrations/background_migrations/backfill_project_issues_count.rb`
+and the regular migration file `db/migrate/xxxxxxxxxxxxxx_enqueue_backfill_project_issues_count.rb` where we enqueue it.
 
 The generated class is a subclass of `OnlineMigrations::BackgroundMigration` that implements:
 
@@ -76,12 +77,16 @@ end
 You can enqueue your background migration to be run by the scheduler via:
 
 ```ruby
-# db/migrate/xxxxxxxxxxxxxx_backfill_project_issues_count.rb
-# ...
-def up
-  enqueue_background_migration("BackfillProjectIssuesCount")
+# db/migrate/xxxxxxxxxxxxxx_enqueue_backfill_project_issues_count.rb
+class EnqueueBackfillProjectIssuesCount < ActiveRecord::Migration[7.1]
+  def up
+    enqueue_background_migration("BackfillProjectIssuesCount")
+  end
+
+  def down
+    remove_background_migration("BackfillProjectIssuesCount")
+  end
 end
-# ...
 ```
 
 `enqueue_background_migration` accepts additional configuration options which controls how the background migration is run. Check the [source code](https://github.com/fatkodima/online_migrations/blob/master/lib/online_migrations/background_migrations/migration_helpers.rb) for the list of all available configuration options.
@@ -106,7 +111,17 @@ end
 And pass them when enqueuing:
 
 ```ruby
-enqueue_background_migration("MyMigrationWithArgs", arg1, arg2, ...)
+def up
+  enqueue_background_migration("MyMigrationWithArgs", arg1, arg2, ...)
+end
+```
+
+Make sure to also pass the arguments inside the `down` method of the migration:
+
+```ruby
+def down
+  remove_background_migration("MyMigrationWithArgs", arg1, arg2, ...)
+end
 ```
 
 ## Considerations when writing Background Migrations

data/docs/configuring.md

@@ -101,13 +101,15 @@ config.lock_retrier = OnlineMigrations::ExponentialLockRetrier.new(
 )
 ```
 
-When statement within transaction fails - the whole transaction is retried.
+When a statement within transaction fails - the whole transaction is retried. If any statement fails when running outside a transaction (e.g. using `disable_ddl_transaction!`) then only that statement is retried.
 
-To permanently disable lock retries, you can set `lock_retrier` to `nil`.
+**Note**: Statements are retried by default, unless lock retries are disabled. It is possible to implement more sophisticated lock retriers. See [source code](https://github.com/fatkodima/online_migrations/blob/master/lib/online_migrations/lock_retrier.rb) for the examples.
 
 To temporarily disable lock retries while running migrations, set `DISABLE_LOCK_RETRIES` env variable. This is useful when you are deploying a hotfix and do not want to wait too long while the lock retrier safely tries to acquire the lock, but try to acquire the lock immediately with the default configured lock timeout value.
 
-**Note**: Statements are retried by default, unless lock retries are disabled. It is possible to implement more sophisticated lock retriers. See [source code](https://github.com/fatkodima/online_migrations/blob/master/lib/online_migrations/lock_retrier.rb) for the examples.
+To permanently disable lock retries, you can set `lock_retrier` to `nil`.
+
+Finally, if your lock retrier implementation does not have an explicit `lock_timeout` value configured, then the timeout behavior will fallback to the database configuration (`config/database.yml`) or the PostgreSQL server config value ([off by default](https://www.postgresql.org/docs/current/runtime-config-client.html#GUC-LOCK-TIMEOUT)). Take care configuring this value, as this fallback may result in your migrations running without a lock timeout!
 
 ## Existing Migrations
 

data/lib/generators/online_migrations/background_migration_generator.rb

@@ -1,12 +1,15 @@
 # frozen_string_literal: true
 
 require "rails/generators"
+require "rails/generators/active_record/migration"
 
 module OnlineMigrations
   # @private
   class BackgroundMigrationGenerator < Rails::Generators::NamedBase
+    include ActiveRecord::Generators::Migration
+
     source_root File.expand_path("templates", __dir__)
-    desc "This generator creates a background migration file."
+    desc "This generator creates a background migration related files."
 
     def create_background_migration_file
       migrations_module_file_path = migrations_module.underscore
@@ -20,6 +23,10 @@ module OnlineMigrations
       template("background_migration.rb", template_file)
     end
 
+    def create_migration_file
+      migration_template("migration.rb", File.join(db_migrate_path, "enqueue_#{file_name}.rb"))
+    end
+
     private
       def migrations_module
         config.migrations_module
@@ -28,5 +35,9 @@ module OnlineMigrations
       def config
         OnlineMigrations.config.background_migrations
       end
+
+      def migration_parent
+        "ActiveRecord::Migration[#{Utils.ar_version}]"
+      end
   end
 end

data/lib/generators/online_migrations/install_generator.rb

@@ -15,7 +15,7 @@ module OnlineMigrations
     end
 
     def create_migration_file
-      migration_template("migration.rb", File.join(db_migrate_path, "install_online_migrations.rb"))
+      migration_template("install_migration.rb", File.join(db_migrate_path, "install_online_migrations.rb"))
     end
 
     private

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

@@ -0,0 +1,51 @@
+class InstallOnlineMigrations < <%= migration_parent %>
+  def change
+    create_table :background_migrations do |t|
+      t.bigint :parent_id
+      t.string :migration_name, null: false
+      t.jsonb :arguments, default: [], null: false
+      t.string :batch_column_name, null: false
+      t.bigint :min_value, null: false
+      t.bigint :max_value, null: false
+      t.bigint :rows_count
+      t.integer :batch_size, null: false
+      t.integer :sub_batch_size, null: false
+      t.integer :batch_pause, null: false
+      t.integer :sub_batch_pause_ms, null: false
+      t.integer :batch_max_attempts, null: false
+      t.string :status, default: "enqueued", null: false
+      t.string :shard
+      t.boolean :composite, default: false, null: false
+      t.timestamps
+
+      t.foreign_key :background_migrations, column: :parent_id, on_delete: :cascade
+
+      t.index [:migration_name, :arguments, :shard],
+        unique: true, name: :index_background_migrations_on_unique_configuration
+    end
+
+    create_table :background_migration_jobs do |t|
+      t.bigint :migration_id, null: false
+      t.bigint :min_value, null: false
+      t.bigint :max_value, null: false
+      t.integer :batch_size, null: false
+      t.integer :sub_batch_size, null: false
+      t.integer :pause_ms, null: false
+      t.datetime :started_at
+      t.datetime :finished_at
+      t.string :status, default: "enqueued", null: false
+      t.integer :max_attempts, null: false
+      t.integer :attempts, default: 0, null: false
+      t.string :error_class
+      t.string :error_message
+      t.string :backtrace, array: true
+      t.timestamps
+
+      t.foreign_key :background_migrations, column: :migration_id, on_delete: :cascade
+
+      t.index [:migration_id, :max_value], name: :index_background_migration_jobs_on_max_value
+      t.index [:migration_id, :status, :updated_at], name: :index_background_migration_jobs_on_updated_at
+      t.index [:migration_id, :finished_at], name: :index_background_migration_jobs_on_finished_at
+    end
+  end
+end

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

@@ -1,51 +1,10 @@
-class InstallOnlineMigrations < <%= migration_parent %>
-  def change
-    create_table :background_migrations do |t|
-      t.bigint :parent_id
-      t.string :migration_name, null: false
-      t.jsonb :arguments, default: [], null: false
-      t.string :batch_column_name, null: false
-      t.bigint :min_value, null: false
-      t.bigint :max_value, null: false
-      t.bigint :rows_count
-      t.integer :batch_size, null: false
-      t.integer :sub_batch_size, null: false
-      t.integer :batch_pause, null: false
-      t.integer :sub_batch_pause_ms, null: false
-      t.integer :batch_max_attempts, null: false
-      t.string :status, default: "enqueued", null: false
-      t.string :shard
-      t.boolean :composite, default: false, null: false
-      t.timestamps
-
-      t.foreign_key :background_migrations, column: :parent_id, on_delete: :cascade
-
-      t.index [:migration_name, :arguments, :shard],
-        unique: true, name: :index_background_migrations_on_unique_configuration
-    end
-
-    create_table :background_migration_jobs do |t|
-      t.bigint :migration_id, null: false
-      t.bigint :min_value, null: false
-      t.bigint :max_value, null: false
-      t.integer :batch_size, null: false
-      t.integer :sub_batch_size, null: false
-      t.integer :pause_ms, null: false
-      t.datetime :started_at
-      t.datetime :finished_at
-      t.string :status, default: "enqueued", null: false
-      t.integer :max_attempts, null: false
-      t.integer :attempts, default: 0, null: false
-      t.string :error_class
-      t.string :error_message
-      t.string :backtrace, array: true
-      t.timestamps
-
-      t.foreign_key :background_migrations, column: :migration_id, on_delete: :cascade
+class Enqueue<%= class_name %> < <%= migration_parent %>
+  def up
+    enqueue_background_migration("<%= class_name %>", ...args)
+  end
 
-      t.index [:migration_id, :max_value], name: :index_background_migration_jobs_on_max_value
-      t.index [:migration_id, :status, :updated_at], name: :index_background_migration_jobs_on_updated_at
-      t.index [:migration_id, :finished_at], name: :index_background_migration_jobs_on_finished_at
-    end
+  def down
+    # Make sure to pass the same arguments as in the "up" method, if any.
+    remove_background_migration("<%= class_name %>", ...args)
   end
 end

data/lib/online_migrations/background_migrations/background_migration_class_validator.rb

@@ -16,13 +16,6 @@ module OnlineMigrations
           return
         end
 
-        if relation.joins_values.present? && !record.batch_column_name.to_s.include?(".")
-          record.errors.add(
-            :batch_column_name,
-            "must be a fully-qualified column if you join a table"
-          )
-        end
-
         if relation.arel.orders.present? || relation.arel.taken.present?
           record.errors.add(
             :migration_name,

data/lib/online_migrations/background_migrations/delete_associated_records.rb

@@ -21,7 +21,7 @@ module OnlineMigrations
       end
 
       def process_batch(relation)
-        relation.delete_all(:delete_all)
+        relation.delete_all
       end
     end
   end

data/lib/online_migrations/background_migrations/migration.rb

@@ -30,7 +30,12 @@ module OnlineMigrations
 
       alias_attribute :name, :migration_name
 
-      enum status: STATUSES.index_with(&:to_s)
+      # Avoid deprecation warnings.
+      if Utils.ar_version >= 7
+        enum :status, STATUSES.index_with(&:to_s)
+      else
+        enum status: STATUSES.index_with(&:to_s)
+      end
 
       belongs_to :parent, class_name: name, optional: true
       has_many :children, class_name: name, foreign_key: :parent_id, dependent: :delete_all

data/lib/online_migrations/background_migrations/migration_helpers.rb

@@ -307,7 +307,7 @@ module OnlineMigrations
       #     perform_action_on_relation_in_background("User", { invite_token: nil }, :generate_invite_token)
       #
       # @note This method is better suited for large tables (10/100s of millions of records).
-      #     For smaller tables it is probably better and easier to directly delete associated records.
+      #     For smaller tables it is probably better and easier to directly perform the action on associated records.
       #
       def perform_action_on_relation_in_background(model_name, conditions, action, updates: nil, **options)
         model_name = model_name.name if model_name.is_a?(Class)
@@ -372,8 +372,7 @@ module OnlineMigrations
       def enqueue_background_migration(migration_name, *arguments, **options)
         migration = create_background_migration(migration_name, *arguments, **options)
 
-        run_inline = OnlineMigrations.config.run_background_migrations_inline
-        if run_inline && run_inline.call
+        if Utils.run_background_migrations_inline?
           runner = MigrationRunner.new(migration)
           runner.run_all_migration_jobs
         end
@@ -381,11 +380,26 @@ module OnlineMigrations
         migration
       end
 
+      # Removes the background migration for the given class name and arguments, if exists.
+      #
+      # @param migration_name [String, Class] Background migration job class name
+      # @param arguments [Array] Extra arguments the migration was originally created with
+      #
+      # @example
+      #   remove_background_migration("BackfillProjectIssuesCount")
+      #
+      def remove_background_migration(migration_name, *arguments)
+        migration_name = migration_name.name if migration_name.is_a?(Class)
+        Migration.for_configuration(migration_name, arguments).delete_all
+      end
+
       # @private
       def create_background_migration(migration_name, *arguments, **options)
         options.assert_valid_keys(:batch_column_name, :min_value, :max_value, :batch_size, :sub_batch_size,
             :batch_pause, :sub_batch_pause_ms, :batch_max_attempts)
 
+        migration_name = migration_name.name if migration_name.is_a?(Class)
+
         migration = Migration.new(
           migration_name: migration_name,
           arguments: arguments,

data/lib/online_migrations/background_migrations/migration_job.rb

@@ -37,9 +37,14 @@ module OnlineMigrations
       scope :except_succeeded, -> { where.not(status: :succeeded) }
       scope :attempts_exceeded, -> { where("attempts >= max_attempts") }
 
-      enum status: STATUSES.index_with(&:to_s)
+      # Avoid deprecation warnings.
+      if Utils.ar_version >= 7
+        enum :status, STATUSES.index_with(&:to_s)
+      else
+        enum status: STATUSES.index_with(&:to_s)
+      end
 
-      delegate :migration_class, :migration_object, :migration_relation, :batch_column_name,
+      delegate :migration_name, :migration_class, :migration_object, :migration_relation, :batch_column_name,
         :arguments, :batch_pause, to: :migration
 
       belongs_to :migration
@@ -68,7 +73,10 @@ module OnlineMigrations
           status: self.class.statuses[:enqueued],
           attempts: 0,
           started_at: nil,
-          finished_at: nil
+          finished_at: nil,
+          error_class: nil,
+          error_message: nil,
+          backtrace: nil
         )
       end
 

data/lib/online_migrations/background_migrations/migration_job_runner.rb

@@ -46,6 +46,7 @@ module OnlineMigrations
         )
 
         ::OnlineMigrations.config.background_migrations.error_handler.call(e, migration_job)
+        raise if Utils.run_background_migrations_inline?
       end
 
       private

data/lib/online_migrations/background_migrations/migration_runner.rb

@@ -51,7 +51,11 @@ module OnlineMigrations
       # @note This method should not be used in production environments
       #
       def run_all_migration_jobs
-        raise "This method is not intended for use in production environments" if !Utils.developer_env?
+        run_inline = OnlineMigrations.config.run_background_migrations_inline
+        if run_inline && !run_inline.call
+          raise "This method is not intended for use in production environments"
+        end
+
         return if migration.completed?
 
         mark_as_running

data/lib/online_migrations/command_checker.rb

@@ -100,7 +100,7 @@ module OnlineMigrations
       end
 
       def set_statement_timeout
-        if !@statement_timeout_set
+        if !defined?(@statement_timeout_set)
           if (statement_timeout = OnlineMigrations.config.statement_timeout)
             # TODO: inline this method call after deprecated `disable_statement_timeout` method removal.
             connection.__set_statement_timeout(statement_timeout)
@@ -527,6 +527,30 @@ module OnlineMigrations
       end
       alias add_belongs_to add_reference
 
+      def add_reference_concurrently(table_name, ref_name, **options)
+        # Always added by default in 5.0+
+        index = options.fetch(:index, true)
+
+        if index.is_a?(Hash) && index[:using].to_s == "hash" && postgresql_version < Gem::Version.new("10")
+          raise_error :add_hash_index
+        end
+
+        foreign_key = options.fetch(:foreign_key, false)
+
+        if foreign_key
+          foreign_table_name = Utils.foreign_table_name(ref_name, options)
+          @foreign_key_tables << foreign_table_name.to_s
+        end
+
+        if !options[:polymorphic]
+          type = (options[:type] || :bigint).to_sym
+          column_name = "#{ref_name}_id"
+
+          foreign_key_options = foreign_key.is_a?(Hash) ? foreign_key : {}
+          check_mismatched_foreign_key_type(table_name, column_name, type, **foreign_key_options)
+        end
+      end
+
       def add_index(table_name, column_name, **options)
         if options[:using].to_s == "hash" && postgresql_version < Gem::Version.new("10")
           raise_error :add_hash_index
@@ -830,8 +854,14 @@ module OnlineMigrations
           if connection.table_exists?(foreign_table_name)
             primary_key = options[:primary_key] || connection.primary_key(foreign_table_name)
             primary_key_column = column_for(foreign_table_name, primary_key)
+            return if primary_key_column.nil?
+
+            primary_key_type = primary_key_column.sql_type.to_sym
+            # Having bigint foreign keys is safe and people should
+            # detect integer primary keys via some other tools.
+            return if type == :bigint && primary_key_type == :integer
 
-            if primary_key_column && type != primary_key_column.sql_type.to_sym
+            if type != primary_key_type
               raise_error :mismatched_foreign_key_type,
                 table_name: table_name, column_name: column_name
             end

data/lib/online_migrations/command_recorder.rb

@@ -26,6 +26,7 @@ module OnlineMigrations
       :add_reference_concurrently,
       :change_column_type_in_background,
       :enqueue_background_migration,
+      :remove_background_migration,
 
       # column type change helpers
       :initialize_column_type_change,
@@ -77,6 +78,10 @@ module OnlineMigrations
 
       include StraightReversions
 
+      def invert_add_reference_concurrently(args)
+        [:remove_reference, args]
+      end
+
       def invert_swap_column_names(args)
         table_name, column1, column2 = args
         [:swap_column_names, [table_name, column2, column1]]

data/lib/online_migrations/database_tasks.rb

@@ -5,7 +5,7 @@ module OnlineMigrations
   module DatabaseTasks
     def migrate(*)
       super
-    rescue => e # rubocop:disable Style/RescueStandardError
+    rescue => e
       if e.cause.is_a?(OnlineMigrations::Error)
         # strip cause
         def e.cause

data/lib/online_migrations/error_messages.rb

@@ -110,7 +110,7 @@ A safer approach is to:
 1. ignore the column:
 
   class <%= model %> < ApplicationRecord
-    self.ignored_columns = [\"<%= column_name %>\"]
+    self.ignored_columns += [\"<%= column_name %>\"]
   end
 
 2. deploy
@@ -151,7 +151,7 @@ It will use a combination of a VIEW and column aliasing to work with both column
 <% if enumerate_columns_in_select_statements %>
 5. Ignore old column
 
-  self.ignored_columns = [:<%= column_name %>]
+  self.ignored_columns += [:<%= column_name %>]
 
 6. Deploy
 7. Remove the column rename config from step 1
@@ -295,10 +295,10 @@ end
 Active Record caches database columns at runtime, so if you drop a column, it can cause exceptions until your app reboots.
 A safer approach is to:
 
-1. Ignore the column(s):
+1. Ignore the column:
 
   class <%= model %> < ApplicationRecord
-    self.ignored_columns = <%= columns %>
+    self.ignored_columns += <%= columns %>
   end
 
 2. Deploy
@@ -310,7 +310,7 @@ A safer approach is to:
     end
   end
 
-4. Remove columns ignoring
+4. Remove column ignoring from step 1
 5. Deploy
 <% end %>",
 

data/lib/online_migrations/lock_retrier.rb

@@ -229,11 +229,9 @@ module OnlineMigrations
     end
 
     def lock_timeout(*)
-      0
     end
 
     def delay(*)
-      0
     end
 
     def with_lock_retries

data/lib/online_migrations/schema_cache.rb

@@ -78,7 +78,7 @@ module OnlineMigrations
   end
 
   # @private
-  module SchemaCache7
+  module SchemaCache71
     # Active Record >= 7.1 changed signature of the methods,
     # see https://github.com/rails/rails/pull/48716.
     def primary_keys(connection, table_name)
@@ -154,4 +154,82 @@ module OnlineMigrations
         columns << new_column.freeze
       end
   end
+
+  # @private
+  module SchemaCache72
+    # Active Record >= 7.2 changed signature of the methods,
+    # see https://github.com/rails/rails/pull/48716.
+    def primary_keys(pool, table_name)
+      if (renamed_table = renamed_table?(pool, table_name))
+        super(pool, renamed_table)
+      elsif renamed_column?(pool, table_name)
+        super(pool, column_rename_table(table_name))
+      else
+        super
+      end
+    end
+
+    def columns(pool, table_name)
+      if (renamed_table = renamed_table?(pool, table_name))
+        super(pool, renamed_table)
+      elsif renamed_column?(pool, table_name)
+        columns = super(pool, column_rename_table(table_name))
+        OnlineMigrations.config.column_renames[table_name].each do |old_column_name, new_column_name|
+          duplicate_column(old_column_name, new_column_name, columns)
+        end
+        columns
+      else
+        super.reject { |column| column.name.end_with?("_for_type_change") }
+      end
+    end
+
+    def indexes(pool, table_name)
+      if (renamed_table = renamed_table?(pool, table_name))
+        super(pool, renamed_table)
+      elsif renamed_column?(pool, table_name)
+        super(pool, column_rename_table(table_name))
+      else
+        super
+      end
+    end
+
+    def clear_data_source_cache!(pool, name)
+      if (renamed_table = renamed_table?(pool, name))
+        super(pool, renamed_table)
+      end
+
+      if renamed_column?(pool, name)
+        super(pool, column_rename_table(name))
+      end
+
+      super
+    end
+
+    private
+      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)
+        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)
+      end
+
+      def column_rename_table(table_name)
+        "#{table_name}_column_rename"
+      end
+
+      def duplicate_column(old_column_name, new_column_name, columns)
+        old_column = columns.find { |column| column.name == old_column_name }
+        new_column = old_column.dup
+        # Active Record defines only reader for :name
+        new_column.instance_variable_set(:@name, new_column_name)
+        # Correspond to the Active Record freezing of each column
+        columns << new_column.freeze
+      end
+  end
 end

data/lib/online_migrations/utils.rb

@@ -151,6 +151,11 @@ module OnlineMigrations
         db_config = ActiveRecord::Base.configurations.configs_for(env_name: env)
         db_config.reject(&:replica?).size > 1
       end
+
+      def run_background_migrations_inline?
+        run_inline = OnlineMigrations.config.run_background_migrations_inline
+        run_inline && run_inline.call
+      end
     end
   end
 end

data/lib/online_migrations/version.rb

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

data/lib/online_migrations.rb

@@ -7,6 +7,7 @@ require "online_migrations/utils"
 require "online_migrations/change_column_type_helpers"
 require "online_migrations/background_migrations/migration_helpers"
 require "online_migrations/schema_statements"
+require "online_migrations/schema_cache"
 require "online_migrations/migration"
 require "online_migrations/migrator"
 require "online_migrations/schema_dumper"
@@ -29,11 +30,6 @@ module OnlineMigrations
   autoload :CommandChecker
   autoload :BackgroundMigration
 
-  autoload_at "online_migrations/schema_cache" do
-    autoload :SchemaCache
-    autoload :SchemaCache7
-  end
-
   autoload_at "online_migrations/lock_retrier" do
     autoload :LockRetrier
     autoload :ConstantLockRetrier
@@ -102,8 +98,10 @@ module OnlineMigrations
       ActiveRecord::Tasks::DatabaseTasks.singleton_class.prepend(OnlineMigrations::DatabaseTasks)
       ActiveRecord::Migration::CommandRecorder.include(OnlineMigrations::CommandRecorder)
 
-      if OnlineMigrations::Utils.ar_version >= 7.1
-        ActiveRecord::ConnectionAdapters::SchemaCache.prepend(OnlineMigrations::SchemaCache7)
+      if OnlineMigrations::Utils.ar_version >= 7.2
+        ActiveRecord::ConnectionAdapters::SchemaCache.prepend(OnlineMigrations::SchemaCache72)
+      elsif OnlineMigrations::Utils.ar_version >= 7.1
+        ActiveRecord::ConnectionAdapters::SchemaCache.prepend(OnlineMigrations::SchemaCache71)
       else
         ActiveRecord::ConnectionAdapters::SchemaCache.prepend(OnlineMigrations::SchemaCache)
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: online_migrations
 version: !ruby/object:Gem::Version
-  version: 0.14.0
+  version: 0.15.0
 platform: ruby
 authors:
 - fatkodima
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-01 00:00:00.000000000 Z
+date: 2024-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -41,6 +41,7 @@ files:
 - lib/generators/online_migrations/templates/add_sharding_to_online_migrations.rb.tt
 - lib/generators/online_migrations/templates/background_migration.rb.tt
 - lib/generators/online_migrations/templates/initializer.rb.tt
+- lib/generators/online_migrations/templates/install_migration.rb.tt
 - lib/generators/online_migrations/templates/migration.rb.tt
 - lib/generators/online_migrations/upgrade_generator.rb
 - lib/online_migrations.rb