pg_ha_migrations 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b472247321efcc716bc1f7c4e497dcbe32058e4d2ce11d8f6d0d9fea7136755
-  data.tar.gz: 820bf96e3200deb2cffd30ba5534513600d05138f57f7812539cc8b33f96058a
+  metadata.gz: 36546429641ce670fa116d251f17993538b3ab60b5014c1917103e45b1fa0f45
+  data.tar.gz: 969078be676bc767aa8f325328da8aeda95b6120e620c5d8556bda639c94a3f4
 SHA512:
-  metadata.gz: b95318b6bf16ea1f5c9ec2514a6d6cccf912fc051caa9a954c58a038431d9afbbf01a23576f4ceaab1856922dc6e9aee5573743abb619f494c0f6023dcb217a9
-  data.tar.gz: 8bf47117312865cf57b51d200f3d131c266171a0f9899f39b8aa3f09aad50842555d93007a0b33a00873a7d67ec56a0f9b9027a028bf28338399939cd285ceda
+  metadata.gz: d2ee13eed929c6278cf745b6b4e4ecff210858e930f97be67d0db2ff4bd2efc65288cc2c80090343cb6cc3e2097c8e1fcf62955fddf442eced723660793706df
+  data.tar.gz: d1f0853d0d47774e889835716f6619097f3e56016e8997a7115e31c34e87dfc80b0871eee128161f4583247265afbfd9aeae752312dad4bb11a5dd5bf0946184

data/.gitignore

@@ -7,6 +7,7 @@
 /pkg/
 /spec/reports/
 /tmp/
+/gemfiles/*.lock
 
 # rspec failure tracking
 .rspec_status

data/.travis.yml

@@ -9,3 +9,8 @@ addons:
 before_install:
   - gem uninstall -v '>= 2' -i $(rvm gemdir)@global -ax bundler || true
   - gem install bundler -v 1.15.4
+gemfile:
+  - gemfiles/rails_5.0.gemfile
+  - gemfiles/rails_5.1.gemfile
+  - gemfiles/rails_5.2.gemfile
+script: "bundle exec rake spec"

data/Appraisals

@@ -0,0 +1,12 @@
+appraise "rails-5.0" do
+  gem "rails", "5.0.7.2"
+end
+
+appraise "rails-5.1" do
+  gem "rails", "5.1.7"
+end
+
+appraise "rails-5.2" do
+  gem "rails", "5.2.3"
+end
+

data/README.md

@@ -127,7 +127,13 @@ unsafe_add_column :table, :column, :type
 Safely change the default value for a column.
 
 ```ruby
+# Constant value:
 safe_change_column_default :table, :column, "value"
+safe_change_column_default :table, :column, DateTime.new(...)
+# Functional expression evaluated at row insert time:
+safe_change_column_default :table, :column, -> { "NOW()" }
+# Functional expression evaluated at migration time:
+safe_change_column_default :table, :column, -> { "'NOW()'" }
 ```
 
 #### safe\_make\_column\_nullable
@@ -168,6 +174,37 @@ Safely remove an index. Migrations that contain this statement must also include
 safe_remove_concurrent_index :table, :name => :index_name
 ```
 
+#### safe\_add\_unvalidated\_check\_constraint
+
+Safely add a `CHECK` constraint. The constraint will not be immediately validated on existing rows to avoid a full table scan while holding an exclusive lock. After adding the constraint, you'll need to use `safe_validate_check_constraint` to validate existing rows.
+
+```ruby
+safe_add_unvalidated_check_constraint :table, "column LIKE 'example%'", name: :constraint_table_on_column_like_example
+```
+
+#### safe\_validate\_check\_constraint
+
+Safely validate (without acquiring an exclusive lock) existing rows for a newly added but as-yet unvalidated `CHECK` constraint.
+
+```ruby
+safe_validate_check_constraint :table, name: :constraint_table_on_column_like_example
+```
+
+#### safe\_rename\_constraint
+
+Safely rename any (not just `CHECK`) constraint.
+
+```ruby
+safe_rename_constraint :table, from: :constraint_table_on_column_like_typo, to: :constraint_table_on_column_like_example
+```
+
+#### unsafe\_remove\_constraint
+
+Drop any (not just `CHECK`) constraint.
+
+```ruby
+unsafe_remove_constraint :table, name: :constraint_table_on_column_like_example
+```
 
 ### Utilities
 

data/Rakefile

@@ -1,8 +1,13 @@
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
+require "appraisal"
 require_relative File.join("lib", "pg_ha_migrations")
 
 RSpec::Core::RakeTask.new(:spec)
 
+if !ENV["APPRAISAL_INITIALIZED"] && !ENV["TRAVIS"]
+    task :default => :appraisal
+end
+
 task :default => :spec
 

data/gemfiles/.bundle/config

@@ -0,0 +1,2 @@
+---
+BUNDLE_RETRY: "1"

data/gemfiles/rails_5.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "5.0.7.2"
+
+gemspec path: "../"

data/gemfiles/rails_5.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "5.1.7"
+
+gemspec path: "../"

data/gemfiles/rails_5.2.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "5.2.3"
+
+gemspec path: "../"

data/lib/pg_ha_migrations/blocking_database_transactions.rb

@@ -1,13 +1,23 @@
 module PgHaMigrations
   class BlockingDatabaseTransactions
-    LongRunningTransaction = Struct.new(:database, :current_query, :transaction_age, :tables_with_locks) do
+    LongRunningTransaction = Struct.new(:database, :current_query, :state, :transaction_age, :tables_with_locks) do
       def description
-        "#{database} | tables (#{tables_with_locks.join(', ')}) have been locked for #{transaction_age} | query: #{current_query}"
+        locked_tables = tables_with_locks.compact
+        [
+          database,
+          locked_tables.size > 0 ? "tables (#{locked_tables.join(', ')})" : nil,
+          "#{idle? ? "currently idle " : ""}transaction open for #{transaction_age}",
+          "#{idle? ? "last " : ""}query: #{current_query}"
+        ].compact.join(" | ")
       end
 
       def concurrent_index_creation?
         !!current_query.match(/create\s+index\s+concurrently/i)
       end
+
+      def idle?
+        state == "idle in transaction"
+      end
     end
 
     def self.autovacuum_regex
@@ -15,7 +25,7 @@ module PgHaMigrations
     end
 
     def self.find_blocking_transactions(minimum_transaction_age = "0 seconds")
-      pid_column, state_column = if ActiveRecord::Base.connection.select_value("SHOW server_version") =~ /9\.1/
+      pid_column, query_column = if ActiveRecord::Base.connection.select_value("SHOW server_version") =~ /9\.1/
         ["procpid", "current_query"]
       else
         ["pid", "query"]
@@ -24,26 +34,32 @@ module PgHaMigrations
       raw_query = <<-SQL
         SELECT
           psa.datname as database, -- Will only ever be one database
-          psa.#{state_column} as current_query,
+          psa.#{query_column} as current_query,
+          psa.state,
           clock_timestamp() - psa.xact_start AS transaction_age,
           array_agg(distinct c.relname) AS tables_with_locks
         FROM pg_stat_activity psa -- Cluster wide
-          JOIN pg_locks l ON (psa.#{pid_column} = l.pid)  -- Cluster wide
-          JOIN pg_class c ON (l.relation = c.oid) -- Database wide
-          JOIN pg_namespace ns ON (c.relnamespace = ns.oid) -- Database wide
-        WHERE psa.#{pid_column} != pg_backend_pid()
-          AND ns.nspname != 'pg_catalog'
-          AND c.relkind = 'r'
-          AND psa.xact_start < clock_timestamp() - ?::interval
-          AND psa.#{state_column} !~ ?
-          AND (
+          LEFT JOIN pg_locks l ON (psa.#{pid_column} = l.pid)  -- Cluster wide
+          LEFT JOIN pg_class c ON ( -- Database wide
+            l.locktype = 'relation'
+            AND l.relation = c.oid
             -- Be explicit about this being for a single database -- it's already implicit in
             -- the relations used, and if we don't restrict this we could get incorrect results
             -- with oid collisions from pg_namespace and pg_class.
-            l.database = 0
-            OR l.database = (SELECT d.oid FROM pg_database d WHERE d.datname = current_database())
+            AND l.database = (SELECT d.oid FROM pg_database d WHERE d.datname = current_database())
           )
-        GROUP BY psa.datname, psa.#{state_column}, psa.xact_start
+          LEFT JOIN pg_namespace ns ON (c.relnamespace = ns.oid) -- Database wide
+        WHERE psa.#{pid_column} != pg_backend_pid()
+          AND (
+            l.locktype != 'relation'
+            OR (
+               ns.nspname != 'pg_catalog'
+               AND c.relkind = 'r'
+            )
+          )
+          AND psa.xact_start < clock_timestamp() - ?::interval
+          AND psa.#{query_column} !~ ?
+        GROUP BY psa.datname, psa.#{query_column}, psa.state, psa.xact_start
       SQL
 
       query = ActiveRecord::Base.send(:sanitize_sql_for_conditions, [raw_query, minimum_transaction_age, autovacuum_regex])

data/lib/pg_ha_migrations/hacks/cleanup_unnecessary_output.rb

@@ -0,0 +1,29 @@
+require "active_record/migration/compatibility"
+
+module PgHaMigrations
+  module ActiveRecordHacks
+    module CleanupUnnecessaryOutput
+      # This is fixed in Rails 6+, but previously there were several
+      # places where #adapter_name was called directly which implicitly
+      # delegated to the connection through #method_missing. That
+      # delegation though results in wrapping the call in #say_with_time
+      # which unnecessarily outputs a bunch of calls to #adapter_name.
+      # The easiest way to clean this up retroactively is to just patch
+      # in a direct dispatch to the connection's method.
+      #
+      # See: https://github.com/rails/rails/commit/eb7c71bcd3d0c7e079dffdb11e43fb466eec06aa
+      def adapter_name
+        connection.adapter_name
+      end
+    end
+  end
+end
+
+patchable_module = [
+ defined?(ActiveRecord::Migration::Compatibility::V5_2) ? ActiveRecord::Migration::Compatibility::V5_2 : nil,
+ defined?(ActiveRecord::Migration::Compatibility::V5_1) ? ActiveRecord::Migration::Compatibility::V5_1 : nil,
+ defined?(ActiveRecord::Migration::Compatibility::V5_0) ? ActiveRecord::Migration::Compatibility::V5_0 : nil,
+].detect { |m| m }
+if patchable_module
+  patchable_module.prepend(PgHaMigrations::ActiveRecordHacks::CleanupUnnecessaryOutput)
+end

data/lib/pg_ha_migrations/safe_statements.rb

@@ -45,10 +45,51 @@ module PgHaMigrations::SafeStatements
   def safe_change_column_default(table_name, column_name, default_value)
     column = connection.send(:column_for, table_name, column_name)
 
+    # In 5.2 we have an edge whereby passing in a string literal with an expression
+    # results in confusing behavior because instead of being executed in the database
+    # that expression is turned into a Ruby nil before being sent to the database layer;
+    # this seems to be an expected side effect of a change that was targeted at a use
+    # case unrelated to migrations: https://github.com/rails/rails/commit/7b2dfdeab6e4ef096e4dc1fe313056f08ccf7dc5
+    #
+    # On the other hand, the behavior in 5.1 is also confusing because it quotes the
+    # expression (instead of maintaining the string as-is), which results in Postgres
+    # evaluating the expression once when executing the DDL and setting the default to
+    # the constant result of that evaluation rather than setting the default to the
+    # expression itself.
+    #
+    # Therefore we want to disallow passing in an expression directly as a string and
+    # require the use of a Proc instead with specific quoting rules to determine exact
+    # behavior. It's fairly difficult (without relying on something like the PgQuery gem
+    # which requires native extensions built with the Postgres dev packages installed)
+    # to determine if a string literal represent an expression or just a constant. So
+    # instead of trying to parse the expression, we employ a set of heuristics:
+    # - If the column is text-like or binary, then we can allow anything in the default
+    #   value since a Ruby string there will always coerce directly to the equivalent
+    #   text/binary value rather than being interpreted as a DDL-time expression.
+    # - Otherwise, disallow any Ruby string values and instead require the Ruby object
+    #   type that maps to the column type.
+    #
+    # These heuristics eliminate (virtually?) all ambiguity. In theory there's a
+    # possiblity that some custom object could be coerced-Ruby side into a SQL string
+    # that does something weird here, but that seems an odd enough case that we can
+    # safely ignore it.
     if default_value.present? &&
        !default_value.is_a?(Proc) &&
-       quote_default_expression(default_value, column) == "NULL"
-      raise PgHaMigrations::InvalidMigrationError, "Requested new default value of <#{default_value}>, but that casts to NULL for the type <#{column.type}>. Did you mean to you mean to use a Proc instead?"
+       (
+         connection.quote_default_expression(default_value, column) == "NULL" ||
+         (
+           ![:string, :text, :binary].include?(column.sql_type_metadata.type) &&
+           default_value.is_a?(String)
+         )
+       )
+      raise PgHaMigrations::InvalidMigrationError, <<~ERROR
+        Setting a default value to an expression using a string literal is ambiguous.
+
+        If you want the default to be:
+        * ...a constant scalar value, use the matching Ruby object type instead of a string if possible (e.g., `DateTime.new(...)`).
+        * ...an expression evaluated at runtime for each row, then pass a Proc that returns the expression string (e.g., `-> { "NOW()" }`).
+        * ...an expression evaluated at migration time, then pass a Proc that returns a quoted expression string (e.g., `-> { "'NOW()'" }`).
+      ERROR
     end
 
     safely_acquire_lock_for_table(table_name) do
@@ -76,7 +117,7 @@ module PgHaMigrations::SafeStatements
     unless options.is_a?(Hash) && options.key?(:name)
       raise ArgumentError, "Expected safe_remove_concurrent_index to be called with arguments (table_name, :name => ...)"
     end
-    unless ActiveRecord::Base.connection.postgresql_version >= 90600
+    unless ActiveRecord::Base.connection.postgresql_version >= 9_06_00
       raise PgHaMigrations::InvalidMigrationError, "Removing an index concurrently is not supported on Postgres 9.1 databases"
     end
     index_size = select_value("SELECT pg_size_pretty(pg_relation_size('#{options[:name]}'))")
@@ -88,6 +129,66 @@ module PgHaMigrations::SafeStatements
     unsafe_execute("SET maintenance_work_mem = '#{PG::Connection.escape_string(gigabytes.to_s)} GB'")
   end
 
+  def safe_add_unvalidated_check_constraint(table, expression, name:)
+    unsafe_add_check_constraint(table, expression, name: name, validate: false)
+  end
+
+  def unsafe_add_check_constraint(table, expression, name:, validate: true)
+    raise ArgumentError, "Expected <name> to be present" unless name.present?
+
+    quoted_table_name = connection.quote_table_name(table)
+    quoted_constraint_name = connection.quote_table_name(name)
+    sql = "ALTER TABLE #{quoted_table_name} ADD CONSTRAINT #{quoted_constraint_name} CHECK (#{expression}) #{validate ? "" : "NOT VALID"}"
+
+    safely_acquire_lock_for_table(table) do
+      say_with_time "add_check_constraint(#{table.inspect}, #{expression.inspect}, name: #{name.inspect}, validate: #{validate.inspect})" do
+        connection.execute(sql)
+      end
+    end
+  end
+
+  def safe_validate_check_constraint(table, name:)
+    raise ArgumentError, "Expected <name> to be present" unless name.present?
+
+    quoted_table_name = connection.quote_table_name(table)
+    quoted_constraint_name = connection.quote_table_name(name)
+    sql = "ALTER TABLE #{quoted_table_name} VALIDATE CONSTRAINT #{quoted_constraint_name}"
+
+    say_with_time "validate_check_constraint(#{table.inspect}, name: #{name.inspect})" do
+      connection.execute(sql)
+    end
+  end
+
+  def safe_rename_constraint(table, from:, to:)
+    raise ArgumentError, "Expected <from> to be present" unless from.present?
+    raise ArgumentError, "Expected <to> to be present" unless to.present?
+
+    quoted_table_name = connection.quote_table_name(table)
+    quoted_constraint_from_name = connection.quote_table_name(from)
+    quoted_constraint_to_name = connection.quote_table_name(to)
+    sql = "ALTER TABLE #{quoted_table_name} RENAME CONSTRAINT #{quoted_constraint_from_name} TO #{quoted_constraint_to_name}"
+
+    safely_acquire_lock_for_table(table) do
+      say_with_time "rename_constraint(#{table.inspect}, from: #{from.inspect}, to: #{to.inspect})" do
+        connection.execute(sql)
+      end
+    end
+  end
+
+  def unsafe_remove_constraint(table, name:)
+    raise ArgumentError, "Expected <name> to be present" unless name.present?
+
+    quoted_table_name = connection.quote_table_name(table)
+    quoted_constraint_name = connection.quote_table_name(name)
+    sql = "ALTER TABLE #{quoted_table_name} DROP CONSTRAINT #{quoted_constraint_name}"
+
+    safely_acquire_lock_for_table(table) do
+      say_with_time "remove_constraint(#{table.inspect}, name: #{name.inspect})" do
+        connection.execute(sql)
+      end
+    end
+  end
+
   def _per_migration_caller
     @_per_migration_caller ||= Kernel.caller
   end
@@ -98,6 +199,14 @@ module PgHaMigrations::SafeStatements
     raise PgHaMigrations::UnsupportedAdapter, "This gem only works with the #{expected_adapter} adapter, found #{actual_adapter} instead" unless actual_adapter == expected_adapter
   end
 
+  def migrate(direction)
+    if respond_to?(:change)
+      raise PgHaMigrations::UnsupportedMigrationError, "Tracking changes for automated rollback is not supported; use explicit #up instead."
+    end
+
+    super(direction)
+  end
+
   def exec_migration(conn, direction)
     _check_postgres_adapter!
     super(conn, direction)
@@ -123,7 +232,7 @@ module PgHaMigrations::SafeStatements
       end
 
       connection.transaction do
-        adjust_timeout_method = connection.postgresql_version >= 90300 ? :adjust_lock_timeout : :adjust_statement_timeout
+        adjust_timeout_method = connection.postgresql_version >= 9_03_00 ? :adjust_lock_timeout : :adjust_statement_timeout
         begin
           method(adjust_timeout_method).call(PgHaMigrations::LOCK_TIMEOUT_SECONDS) do
             connection.execute("LOCK #{quoted_table_name};")

data/lib/pg_ha_migrations/unsafe_statements.rb

@@ -43,6 +43,7 @@ module PgHaMigrations::UnsafeStatements
   delegate_unsafe_method_to_migration_base_class :execute
   delegate_unsafe_method_to_migration_base_class :remove_index
   delegate_unsafe_method_to_migration_base_class :add_foreign_key
+  delegate_unsafe_method_to_migration_base_class :remove_foreign_key
 
   disable_or_delegate_default_method :create_table, ":create_table is NOT SAFE! Use safe_create_table instead"
   disable_or_delegate_default_method :add_column, ":add_column is NOT SAFE! Use safe_add_column instead"
@@ -58,6 +59,7 @@ module PgHaMigrations::UnsafeStatements
   disable_or_delegate_default_method :execute, ":execute is NOT SAFE! Explicitly call :unsafe_execute to proceed", allow_reentry_from_compatibility_module: true
   disable_or_delegate_default_method :remove_index, ":remove_index is NOT SAFE! Use safe_remove_concurrent_index instead for Postgres 9.6 databases; Explicitly call :unsafe_remove_index to proceed on Postgres 9.1"
   disable_or_delegate_default_method :add_foreign_key, ":add_foreign_key is NOT SAFE! Explicitly call :unsafe_add_foreign_key"
+  disable_or_delegate_default_method :remove_foreign_key, ":remove_foreign_key is NOT SAFE! Explicitly call :unsafe_remove_foreign_key"
 
   def unsafe_create_table(table, options={}, &block)
     if options[:force] && !PgHaMigrations.config.allow_force_create_table

data/lib/pg_ha_migrations/version.rb

@@ -1,3 +1,3 @@
 module PgHaMigrations
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/lib/pg_ha_migrations.rb

@@ -35,6 +35,10 @@ module PgHaMigrations
   # as expected or get the schema into an inconsistent state
   InvalidMigrationError = Class.new(Exception)
 
+  # Unsupported migrations use ActiveRecord::Migration features that
+  # we don't support, and therefore will likely have unexpected behavior.
+  UnsupportedMigrationError = Class.new(Exception)
+
   # This gem only supports the PostgreSQL adapter at this time.
   UnsupportedAdapter = Class.new(Exception)
 end
@@ -47,6 +51,7 @@ require "pg_ha_migrations/dependent_objects_checks"
 require "pg_ha_migrations/allowed_versions"
 require "pg_ha_migrations/railtie"
 require "pg_ha_migrations/hacks/disable_ddl_transaction"
+require "pg_ha_migrations/hacks/cleanup_unnecessary_output"
 
 module PgHaMigrations::AutoIncluder
   def inherited(klass)

data/pg_ha_migrations.gemspec

@@ -29,13 +29,13 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.15"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency "pg"
   spec.add_development_dependency "db-query-matchers", "~> 0.9.0"
-  spec.add_development_dependency 'pry'
-  spec.add_development_dependency 'pry-byebug'
+  spec.add_development_dependency "pry"
+  spec.add_development_dependency "pry-byebug"
+  spec.add_development_dependency "appraisal", "~> 2.2.0"
 
   spec.add_dependency "rails", ">= 5.0", "< 5.3"
   spec.add_dependency "relation_to_struct"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pg_ha_migrations
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - celeen
@@ -14,22 +14,8 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-09-05 00:00:00.000000000 Z
+date: 2020-01-31 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.15'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.15'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -114,6 +100,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.2.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.2.0
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -161,6 +161,7 @@ files:
 - ".rspec"
 - ".ruby-version"
 - ".travis.yml"
+- Appraisals
 - CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.txt
@@ -168,11 +169,16 @@ files:
 - Rakefile
 - bin/console
 - bin/setup
+- gemfiles/.bundle/config
+- gemfiles/rails_5.0.gemfile
+- gemfiles/rails_5.1.gemfile
+- gemfiles/rails_5.2.gemfile
 - lib/pg_ha_migrations.rb
 - lib/pg_ha_migrations/allowed_versions.rb
 - lib/pg_ha_migrations/blocking_database_transactions.rb
 - lib/pg_ha_migrations/blocking_database_transactions_reporter.rb
 - lib/pg_ha_migrations/dependent_objects_checks.rb
+- lib/pg_ha_migrations/hacks/cleanup_unnecessary_output.rb
 - lib/pg_ha_migrations/hacks/disable_ddl_transaction.rb
 - lib/pg_ha_migrations/railtie.rb
 - lib/pg_ha_migrations/safe_statements.rb
@@ -199,7 +205,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Enforces DDL/migration safety in Ruby on Rails project with an emphasis on