vitess-activerecord-migration 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e4e6886e3311d81a26675589df0420f5e4ea72ef3f5b7a4288029ef2feb4542f
-  data.tar.gz: 407235b276dc0f370663348491e03e2cfa6a2ae7b108c67fbd3f66967162561e
+  metadata.gz: ccf02af6a599148167a09bb0281fcb891663bf455b280c74bcd0f344d15b0008
+  data.tar.gz: 6fbd1977dd22de177572f0de28dbbd80824b445361c62f0d2f1f7741c8b884ba
 SHA512:
-  metadata.gz: d867e55428ac7f48b5a87c34faece5d2de9deab0aa63779b759917e52becb7361cd2c13e8cafb2765008238df62c910392843aa1e2da8795a27c41f97d03820b
-  data.tar.gz: 115075f6113b7b0cf64d2a52b2148048271130c7ded01cdb6ef2367c25b9307b047f3c9ab5990b386f88d9829a816cd765da17d2c318ed99e3e1c97ba385b675
+  metadata.gz: a02873ccab767abf06dc40277d4f75dd169449d43d1e3b40b08c15ddc099efd89edab6016af640379842c92e1d999e1e112c21a97107eda80b5c0858077f9116
+  data.tar.gz: 42336d57e7c0e870a2dae788ab66ed56858683738df9b76094ce87c8df8b09d8844fc878a9679028a9b1de3cc68dde3b37e890b96a5455ef04d56f06f36c9c02

data/.env

@@ -1,3 +1,3 @@
 RUBY_VERSION=${RUBY_VERSION:-3.2}
 RAILS_VERSION=${RAILS_VERSION:-~> 7.2.0}
-VITESS_VERSION=${VITESS_VERSION:-20.0}
+VITESS_VERSION=${VITESS_VERSION:-21.0}

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## [Unreleased]
 
+## [0.3.0] - 2025-08-30
+
+- **BREAKING**: Drop Rails 6.1 support, require ActiveRecord >= 7.0 (#7)
+- **BREAKING**: Raise errors on failed/cancelled/timed-out migrations (#5)
+
 ## [0.1.0] - 2024-09-02
 
 - Initial release

data/README.md

@@ -1,5 +1,9 @@
 # Vitess::Activerecord::Migration
 
+[![Ruby](https://github.com/yoheimuta/vitess-activerecord-migration/actions/workflows/main.yml/badge.svg)](https://github.com/yoheimuta/vitess-activerecord-migration/actions/workflows/main.yml)
+[![Gem Version](https://badge.fury.io/rb/vitess-activerecord-migration.svg)](https://badge.fury.io/rb/vitess-activerecord-migration)
+[![License](http://img.shields.io/:license-mit-blue.svg)](https://github.com/yoheimuta/protolint/blob/master/LICENSE)
+
 vitess-activerecord-migration is a gem for integrating Vitess with ActiveRecord migrations.
 
 Vitess::Activerecord::Migration provides tools to integrate Vitess with ActiveRecord migrations, 

data/lib/vitess/activerecord/migration/version.rb

@@ -3,7 +3,7 @@
 module Vitess
   module Activerecord
     module Migration
-      VERSION = "0.2.0"
+      VERSION = "0.3.0"
     end
   end
 end

data/lib/vitess/activerecord/migration.rb

@@ -7,6 +7,9 @@ module Vitess
   module Activerecord
     module Migration
       class Error < StandardError; end
+      class Failed < Error; end
+      class Cancelled < Error; end
+      class TimedOut < Error; end
 
       # Returns the default DDL strategy.
       # This method is called and set before executing the change, up, or down methods.
@@ -17,13 +20,24 @@ module Vitess
       end
 
       # Returns the timeout seconds for waiting for the completion of the DDL statement.
-      # Even after the timeout, the migration on Vitess will not be stopped, but a warning will be logged.
+      # If the DDL exceeds the timeout:
+      #
+      # * A warning will be logged.
+      # * An error will be raised and halt the Rails migration if `raise_on_timeout` is `true`.
+      # * The Vitess migration will continue.
       #
       # Override this method if you want to change the default timeout.
       def wait_timeout_seconds
         7200 # 120 minutes
       end
 
+      # Returns whether an error will be raised when the DDL statement exceeds wait_timeout_seconds.
+      #
+      # Override this method if you want to change how timeouts are handled.
+      def raise_on_timeout
+        true
+      end
+
       # Returns the columns of SHOW VITESS_MIGRATIONS to log during the run.
       #
       # Override this method if you want to change the default columns.
@@ -114,7 +128,18 @@ module Vitess
         @stopped_uuid ||= []
 
         loop do
+          # A Rails migration will create a separate Vitess migration for each DDL submitted by Rails.
+          # Each Rails migration has a unique version and this is used to group the resulting Vitess migrations using the `migration_context` field.
+          # Vitess records the DDL submitted by Rails in the `migration_statement` column.
+          # If the Rails migration fails and is later retried, Vitess will create additional Vitess migrations with the same `migration_context` from the previous attempt.
+          # Vitess will immediately mark "duplicate" DDLs that were previously successful as complete.
+          # Vitess will retry "duplicate" DDLs that were previously unsuccessful.
+          # The overall Rails migration status is determined by the `migration_status` of the last Vitess migration of each DDL.
+          # For details on Vitess duplicate migration detection, see:
+          # https://vitess.io/docs/22.0/user-guides/schema-changes/advanced-usage/
           migrations = ActiveRecord::Base.connection.select_all("SHOW VITESS_MIGRATIONS LIKE '#{@migration_context}'")
+            .group_by { |migration| migration['migration_statement'] }
+            .map { |_, migrations| migrations.last }
 
           migrations.each do |migration|
             id = migration["id"]
@@ -130,8 +155,10 @@ module Vitess
                 Rails.logger.info("Vitess Migration #{id} completed successfully at #{migration["completed_timestamp"]}")
               when "failed"
                 Rails.logger.error("Vitess Migration #{id} failed: #{migration["message"]} at #{migration["completed_timestamp"]}")
+                raise Failed, "Vitess Migration #{id} failed: #{migration["message"]}"
               when "cancelled"
                 Rails.logger.warn("Vitess Migration #{id} was cancelled at #{migration["cancelled_timestamp"]}")
+                raise Cancelled, "Vitess Migration #{id} was cancelled: #{migration["message"]}"
               end
               @stopped_uuid << id
             else
@@ -146,6 +173,7 @@ module Vitess
 
           if Time.now - start_time > timeout_seconds
             Rails.logger.warn("Vitess Migration did not complete within #{timeout_seconds} seconds. Timing out.")
+            raise TimedOut, "Vitess Migration did not complete within #{timeout_seconds} seconds."
             break
           end
 
@@ -154,6 +182,8 @@ module Vitess
           interval_seconds = [interval_seconds * 2, max_interval_seconds].min
         end
       rescue => e
+        raise e if [Failed, Cancelled].include?(e.class)
+        raise e if e.class == TimedOut && raise_on_timeout
         Rails.logger.error("An error occurred while waiting for Vitess DDL: #{e.message}")
         Rails.logger.error(e.backtrace.join("\n"))
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vitess-activerecord-migration
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Yohei Yoshimuta
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-10-15 00:00:00.000000000 Z
+date: 2025-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,20 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.1'
+        version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.1'
+        version: '7.0'
 description: Vitess::Activerecord::Migration provides tools to integrate Vitess with
   ActiveRecord migrations, allowing for seamless database schema changes in a Vitess
   environment.
@@ -57,7 +51,6 @@ files:
 - sig/rails_support.rbs
 - sig/vitess/activerecord/migration.rbs
 - tmp/.keep
-- vitess-activerecord-migration.gemspec
 homepage: https://github.com/yoheimuta/vitess-activerecord-migration
 licenses:
 - MIT
@@ -66,7 +59,7 @@ metadata:
   homepage_uri: https://github.com/yoheimuta/vitess-activerecord-migration
   source_code_uri: https://github.com/yoheimuta/vitess-activerecord-migration
   changelog_uri: https://github.com/yoheimuta/vitess-activerecord-migration/blob/main/CHANGELOG.md
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -81,8 +74,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3.1
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: A gem for integrating Vitess with ActiveRecord migrations.
 test_files: []

data/vitess-activerecord-migration.gemspec

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "lib/vitess/activerecord/migration/version"
-
-Gem::Specification.new do |spec|
-  spec.name = "vitess-activerecord-migration"
-  spec.version = Vitess::Activerecord::Migration::VERSION
-  spec.authors = ["Yohei Yoshimuta"]
-  spec.email = ["yoheimuta@gmail.com"]
-
-  spec.summary = "A gem for integrating Vitess with ActiveRecord migrations."
-  spec.description = "Vitess::Activerecord::Migration provides tools to integrate Vitess with ActiveRecord migrations, allowing for seamless database schema changes in a Vitess environment."
-  spec.homepage = "https://github.com/yoheimuta/vitess-activerecord-migration"
-  spec.license = "MIT"
-  spec.required_ruby_version = ">= 2.7.0"
-
-  spec.metadata["allowed_push_host"] = "https://rubygems.org"
-
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = "https://github.com/yoheimuta/vitess-activerecord-migration"
-  spec.metadata["changelog_uri"] = "https://github.com/yoheimuta/vitess-activerecord-migration/blob/main/CHANGELOG.md"
-
-  # Specify which files should be added to the gem when it is released.
-  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files = Dir.chdir(__dir__) do
-    `git ls-files -z`.split("\x0").reject do |f|
-      (File.expand_path(f) == __FILE__) ||
-        f.start_with?(*%w[bin/ test/ spec/ features/ .git .circleci appveyor Gemfile])
-    end
-  end
-  spec.bindir = "exe"
-  spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  # Uncomment to register a new dependency of your gem
-  spec.add_dependency "activerecord", ">= 6", "< 8.1"
-
-  # For more information and examples about making a new gem, check out our
-  # guide at: https://bundler.io/guides/creating_gem.html
-end