pg_online_schema_change 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b06d39169d55c016d78969968c71cac7f0c49a3c716d0bc8c5004905ef820fb0
-  data.tar.gz: 78fcc0a928a5efa645ec201cfca093f4ef078b8d683f0776a7f9ad91bb11ef99
+  metadata.gz: 8ef1069f7d159544838ec27cf518f98eb31609a0a2e1878ebd40246b10a6b534
+  data.tar.gz: 8e83cbac78164fb10870537bf4867020eae7cf06f4cd91930a2c8846fbc76c7c
 SHA512:
-  metadata.gz: 179fcaf09bcc36b69186ba48494835841a88c095e010756f0e6f322a146279d0790403f2c195f23f483e89a1cb74c10b50a3ceee26ad6d311e3d9977af1b8c07
-  data.tar.gz: 157255f9d901ec2cb980d84bd664cc0eaae32854bdd79ebd262b2112eef965f8dc2d123277a35446fefd6ecdac4a4ba4c94769589461053241d1eedbe40efa52
+  metadata.gz: 2657c94b3b07730aa3bb282f34cd1ee95f0549d3649b39a61be3ce1b6d2ba5832a1914b6a9788778c912464cbd9773b5e967ed4e6c896127d60e62a0daa76c99
+  data.tar.gz: 4788523d691c25045b7f3c539f9743d880fabcbb01887ae34ec55a6c8a829804a9f4d43f619e79e0c44b1c9fff6ea88361fb5ac1111bc1f491aa17e6e43336d0

data/.rubocop.yml

@@ -1,3 +1,5 @@
+inherit_from: .rubocop_todo.yml
+
 require:
   - rubocop-rspec
   - rubocop-packaging
@@ -14,71 +16,70 @@ AllCops:
     - "vendor/**/*"
 
 Layout/HashAlignment:
-  EnforcedColonStyle:
-    - table
-    - key
-  EnforcedHashRocketStyle:
-    - table
-    - key
+  EnforcedColonStyle: key
+  EnforcedHashRocketStyle:  key
 
 Layout/SpaceAroundEqualsInParameterDefault:
-  EnforcedStyle: no_space
+  EnforcedStyle: space
 
 Metrics/AbcSize:
-  Max: 20
+  Enabled: true
+  Max: 40
   Exclude:
-    - "test/**/*"
+    - "spec/**/*"
 
 Metrics/BlockLength:
+  Max: 100
   Exclude:
     - "*.gemspec"
     - "Rakefile"
+    - "spec/**/*"
 
 Metrics/ClassLength:
   Exclude:
     - "test/**/*"
 
 Metrics/MethodLength:
-  Max: 18
+  Max: 30
   Exclude:
     - "test/**/*"
 
 Metrics/ParameterLists:
-  Max: 6
+  Max: 5
 
 Naming/MemoizedInstanceVariableName:
-  Enabled: false
+  Enabled: true
 
 Naming/VariableNumber:
-  Enabled: false
-
-Rake/Desc:
-  Enabled: false
+  Enabled: true
 
 Style/BarePercentLiterals:
   EnforcedStyle: percent_q
 
 Style/ClassAndModuleChildren:
-  Enabled: false
+  Enabled: true
 
 Style/Documentation:
   Enabled: false
 
 Style/DoubleNegation:
-  Enabled: false
+  Enabled: true
 
 Style/EmptyMethod:
-  Enabled: false
+  Enabled: true
 
 Style/FrozenStringLiteralComment:
-  Enabled: false
+  Enabled: true
 
 Style/NumericPredicate:
-  Enabled: false
+  Enabled: true
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes
 
+Style/StringLiteralsInInterpolation:
+  EnforcedStyle: double_quotes
+
 Style/TrivialAccessors:
   AllowPredicates: true
 
@@ -91,9 +92,41 @@ Style/TrailingCommaInArrayLiteral:
 Style/TrailingCommaInHashLiteral:
   EnforcedStyleForMultiline: comma
 
-Style/SpaceAroundEqualsInParameterDefault:
-  EnforcedStyle: space
+Layout/MultilineArrayBraceLayout:
+  Enabled: true
+  EnforcedStyle: symmetrical
 
-Style/MultilineHashBraceLayout:
+Layout/MultilineHashBraceLayout:
   Enabled: true
   EnforcedStyle: symmetrical
+
+Layout/MultilineAssignmentLayout:
+  Enabled: true
+  EnforcedStyle: same_line
+
+Layout/FirstArrayElementIndentation:
+  Enabled: true
+  EnforcedStyle: consistent
+
+Layout/FirstHashElementIndentation:
+  Enabled: true
+  EnforcedStyle: consistent
+
+Layout/MultilineHashKeyLineBreaks:
+  Enabled: true
+
+Layout/LineLength:
+  Enabled: true
+  Max: 250
+
+Style/FormatStringToken:
+  Enabled: true
+  EnforcedStyle: template
+
+RSpec/MessageSpies:
+  Enabled: true
+  EnforcedStyle: receive
+
+RSpec/FilePath:
+  Enabled: true
+  SpecSuffixOnly: true

data/.rubocop_todo.yml

@@ -0,0 +1,44 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2022-02-21 22:46:44 UTC using RuboCop version 1.23.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 2
+# Configuration parameters: CountComments, CountAsOne.
+Metrics/ClassLength:
+  Max: 233
+
+# Offense count: 2
+# Configuration parameters: IgnoredMethods.
+Metrics/CyclomaticComplexity:
+  Max: 15
+
+# Offense count: 2
+# Configuration parameters: IgnoredMethods.
+Metrics/PerceivedComplexity:
+  Max: 13
+
+# Offense count: 1
+Packaging/GemspecGit:
+  Exclude:
+    - 'pg_online_schema_change.gemspec'
+
+# Offense count: 62
+# Configuration parameters: CountAsOne.
+RSpec/ExampleLength:
+  Max: 55
+
+# Offense count: 38
+RSpec/MultipleExpectations:
+  Max: 14
+
+# Offense count: 6
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: respond_to_missing?
+Style/OptionalBooleanParameter:
+  Exclude:
+    - 'lib/pg_online_schema_change/query.rb'
+    - 'lib/pg_online_schema_change/replay.rb'

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [0.3.0] - 2022-02-21
+
+- Explicitly call dependencies and bump dependencies by @shayonj https://github.com/shayonj/pg-osc/pull/44
+- Introduce Dockerfile and release process https://github.com/shayonj/pg-osc/pull/45
+
 ## [0.2.0] - 2022-02-17
 
 - Use ISOLATION LEVEL SERIALIZABLE ([#42](https://github.com/shayonj/pg-osc/pull/42)) (props to @jfrost)

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pg_online_schema_change (0.2.0)
+    pg_online_schema_change (0.3.0)
       ougai (~> 2.0.0)
       pg (~> 1.3.2)
       pg_query (~> 2.1.3)

data/lib/pg_online_schema_change/cli.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "thor"
 
 module PgOnlineSchemaChange
@@ -25,7 +27,7 @@ module PgOnlineSchemaChange
     def perform
       client_options = Struct.new(*options.keys.map(&:to_sym)).new(*options.values)
 
-      PgOnlineSchemaChange.logger = client_options.verbose
+      PgOnlineSchemaChange.logger(verbose: client_options.verbose)
       PgOnlineSchemaChange::Orchestrate.run!(client_options)
     end
 

data/lib/pg_online_schema_change/client.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pg"
 
 module PgOnlineSchemaChange
@@ -16,7 +18,9 @@ module PgOnlineSchemaChange
       @drop = options.drop
       @kill_backends = options.kill_backends
       @wait_time_for_lock = options.wait_time_for_lock
+
       handle_copy_statement(options.copy_statement)
+      handle_validations
 
       @connection = PG.connect(
         dbname: @dbname,
@@ -26,17 +30,19 @@ module PgOnlineSchemaChange
         port: @port,
       )
 
-      raise Error, "Not a valid ALTER statement: #{@alter_statement}" unless Query.alter_statement?(@alter_statement)
-
-      unless Query.same_table?(@alter_statement)
-        raise Error "All statements should belong to the same table: #{@alter_statement}"
-      end
-
       @table = Query.table(@alter_statement)
 
       PgOnlineSchemaChange.logger.debug("Connection established")
     end
 
+    def handle_validations
+      raise Error, "Not a valid ALTER statement: #{@alter_statement}" unless Query.alter_statement?(@alter_statement)
+
+      return if Query.same_table?(@alter_statement)
+
+      raise Error "All statements should belong to the same table: #{@alter_statement}"
+    end
+
     def handle_copy_statement(statement)
       return if statement.nil? || statement == ""
 

data/lib/pg_online_schema_change/functions.rb

@@ -1,4 +1,6 @@
-FUNC_FIX_SERIAL_SEQUENCE = <<~SQL.freeze
+# frozen_string_literal: true
+
+FUNC_FIX_SERIAL_SEQUENCE = <<~SQL
     CREATE OR REPLACE FUNCTION fix_serial_sequence(_table regclass, _newtable text)
     RETURNS void AS
     $func$
@@ -35,7 +37,7 @@ FUNC_FIX_SERIAL_SEQUENCE = <<~SQL.freeze
   $func$  LANGUAGE plpgsql VOLATILE;
 SQL
 
-FUNC_CREATE_TABLE_ALL = <<~SQL.freeze
+FUNC_CREATE_TABLE_ALL = <<~SQL
   CREATE OR REPLACE FUNCTION create_table_all(source_table text, newsource_table text)
     RETURNS void language plpgsql
     as $$

data/lib/pg_online_schema_change/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module PgOnlineSchemaChange
   module Helper
     def primary_key
@@ -15,7 +17,14 @@ module PgOnlineSchemaChange
       result = Store.send(:get, method)
       return result if result
 
-      raise ArgumentError, "Method `#{method}` doesn't exist."
+      super
+    end
+
+    def respond_to_missing?(method_name, *args)
+      result = Store.send(:get, method)
+      return true if result
+
+      super
     end
   end
 end

data/lib/pg_online_schema_change/orchestrate.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 require "securerandom"
 
 module PgOnlineSchemaChange
   class Orchestrate
-    SWAP_STATEMENT_TIMEOUT = "5s".freeze
+    SWAP_STATEMENT_TIMEOUT = "5s"
 
     extend Helper
 
@@ -21,12 +23,20 @@ module PgOnlineSchemaChange
         Query.run(client.connection, FUNC_FIX_SERIAL_SEQUENCE)
         Query.run(client.connection, FUNC_CREATE_TABLE_ALL)
 
+        setup_store
+      end
+
+      def setup_store
         # Set this early on to ensure their creation and cleanup (unexpected)
         # happens at all times. IOW, the calls from Store.get always return
         # the same value.
         Store.set(:old_primary_table, "pgosc_op_table_#{client.table}")
-        Store.set(:audit_table, "pgosc_at_#{client.table}_#{random_string}")
-        Store.set(:shadow_table, "pgosc_st_#{client.table}_#{random_string}")
+        Store.set(:audit_table, "pgosc_at_#{client.table}_#{pgosc_identifier}")
+        Store.set(:operation_type_column, "operation_type_#{pgosc_identifier}")
+        Store.set(:trigger_time_column, "trigger_time_#{pgosc_identifier}")
+        Store.set(:audit_table_pk, "at_#{pgosc_identifier}_id")
+        Store.set(:audit_table_pk_sequence, "#{audit_table}_#{audit_table_pk}_seq")
+        Store.set(:shadow_table, "pgosc_st_#{client.table}_#{pgosc_identifier}")
       end
 
       def run!(options)
@@ -70,7 +80,7 @@ module PgOnlineSchemaChange
         reader = setup_signals!
         signal = reader.gets.chomp
 
-        while !reader.closed? && IO.select([reader])
+        while !reader.closed? && IO.select([reader]) # rubocop:disable Lint/UnreachableLoop
           logger.info "Signal #{signal} received, cleaning up"
 
           client.connection.cancel
@@ -85,7 +95,7 @@ module PgOnlineSchemaChange
         logger.info("Setting up audit table", { audit_table: audit_table })
 
         sql = <<~SQL
-          CREATE TABLE #{audit_table} (operation_type text, trigger_time timestamp, LIKE #{client.table});
+          CREATE TABLE #{audit_table} (#{audit_table_pk} SERIAL PRIMARY KEY, #{operation_type_column} text, #{trigger_time_column} timestamp, LIKE #{client.table});
         SQL
 
         Query.run(client.connection, sql)
@@ -109,13 +119,13 @@ module PgOnlineSchemaChange
           $$
           BEGIN
             IF ( TG_OP = 'INSERT') THEN
-              INSERT INTO \"#{audit_table}\" select 'INSERT', now(), NEW.* ;
+              INSERT INTO \"#{audit_table}\" select nextval(\'#{audit_table_pk_sequence}\'), 'INSERT', clock_timestamp(), NEW.* ;
               RETURN NEW;
             ELSIF ( TG_OP = 'UPDATE') THEN
-              INSERT INTO \"#{audit_table}\" select 'UPDATE', now(),  NEW.* ;
+              INSERT INTO \"#{audit_table}\" select nextval(\'#{audit_table_pk_sequence}\'), 'UPDATE', clock_timestamp(),  NEW.* ;
               RETURN NEW;
             ELSIF ( TG_OP = 'DELETE') THEN
-              INSERT INTO \"#{audit_table}\" select 'DELETE', now(), OLD.* ;
+              INSERT INTO \"#{audit_table}\" select nextval(\'#{audit_table_pk_sequence}\'), 'DELETE', clock_timestamp(), OLD.* ;
               RETURN NEW;
             END IF;
           END;
@@ -153,7 +163,7 @@ module PgOnlineSchemaChange
         # re-uses transaction with serializable
         # Disabling vacuum to avoid any issues during the process
         result = Query.storage_parameters_for(client, client.table, true) || ""
-        primary_table_storage_parameters = Store.set(:primary_table_storage_parameters, result)
+        Store.set(:primary_table_storage_parameters, result)
 
         logger.debug("Disabling vacuum on shadow and audit table",
                      { shadow_table: shadow_table, audit_table: audit_table })
@@ -185,8 +195,7 @@ module PgOnlineSchemaChange
         # Begin the process to copy data into copy table
         # depending on the size of the table, this can be a time
         # taking operation.
-        logger.info("Clearing contents of audit table before copy..",
-                    { shadow_table: shadow_table, parent_table: client.table })
+        logger.info("Clearing contents of audit table before copy..", { shadow_table: shadow_table, parent_table: client.table })
         Query.run(client.connection, "DELETE FROM #{audit_table}", true)
 
         logger.info("Copying contents..", { shadow_table: shadow_table, parent_table: client.table })
@@ -272,8 +281,10 @@ module PgOnlineSchemaChange
         Query.run(client.connection, sql)
       end
 
-      private def random_string
-        @random_string ||= SecureRandom.hex(3)
+      private
+
+      def pgosc_identifier
+        @pgosc_identifier ||= SecureRandom.hex(3)
       end
     end
   end

data/lib/pg_online_schema_change/query.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "pg_query"
 require "pg"
 
@@ -5,7 +7,7 @@ module PgOnlineSchemaChange
   class Query
     extend Helper
 
-    INDEX_SUFFIX = "_pgosc".freeze
+    INDEX_SUFFIX = "_pgosc"
     DROPPED_COLUMN_TYPE = :AT_DropColumn
     RENAMED_COLUMN_TYPE = :AT_RenameColumn
     LOCK_ATTEMPT = 4
@@ -15,28 +17,28 @@ module PgOnlineSchemaChange
         PgQuery.parse(query).tree.stmts.all? do |statement|
           statement.stmt.alter_table_stmt.instance_of?(PgQuery::AlterTableStmt) || statement.stmt.rename_stmt.instance_of?(PgQuery::RenameStmt)
         end
-      rescue PgQuery::ParseError => e
+      rescue PgQuery::ParseError
         false
       end
 
       def same_table?(query)
-        tables = PgQuery.parse(query).tree.stmts.map do |statement|
+        tables = PgQuery.parse(query).tree.stmts.filter_map do |statement|
           if statement.stmt.alter_table_stmt.instance_of?(PgQuery::AlterTableStmt)
             statement.stmt.alter_table_stmt.relation.relname
           elsif statement.stmt.rename_stmt.instance_of?(PgQuery::RenameStmt)
             statement.stmt.rename_stmt.relation.relname
           end
-        end.compact
+        end
 
         tables.uniq.count == 1
-      rescue PgQuery::ParseError => e
+      rescue PgQuery::ParseError
         false
       end
 
       def table(query)
-        from_rename_statement = PgQuery.parse(query).tree.stmts.map do |statement|
+        from_rename_statement = PgQuery.parse(query).tree.stmts.filter_map do |statement|
           statement.stmt.rename_stmt&.relation&.relname
-        end.compact[0]
+        end[0]
         PgQuery.parse(query).tables[0] || from_rename_statement
       end
 
@@ -48,7 +50,7 @@ module PgOnlineSchemaChange
         connection.async_exec("BEGIN;")
 
         result = connection.async_exec(query, &block)
-      rescue Exception
+      rescue Exception # rubocop:disable Lint/RescueException
         connection.cancel if connection.transaction_status != PG::PQTRANS_IDLE
         connection.block
         logger.info("Exception raised, rolling back query", { rollback: true, query: query })
@@ -144,11 +146,11 @@ module PgOnlineSchemaChange
         end
 
         references.map do |row|
-          if row["definition"].end_with?("NOT VALID")
-            add_statement = "ALTER TABLE #{row["table_on"]} ADD CONSTRAINT #{row["constraint_name"]} #{row["definition"]};"
-          else
-            add_statement = "ALTER TABLE #{row["table_on"]} ADD CONSTRAINT #{row["constraint_name"]} #{row["definition"]} NOT VALID;"
-          end
+          add_statement = if row["definition"].end_with?("NOT VALID")
+                            "ALTER TABLE #{row["table_on"]} ADD CONSTRAINT #{row["constraint_name"]} #{row["definition"]};"
+                          else
+                            "ALTER TABLE #{row["table_on"]} ADD CONSTRAINT #{row["constraint_name"]} #{row["definition"]} NOT VALID;"
+                          end
 
           drop_statement = "ALTER TABLE #{row["table_on"]} DROP CONSTRAINT #{row["constraint_name"]};"
 
@@ -291,7 +293,7 @@ module PgOnlineSchemaChange
           client.connection.quote_ident(select_column)
         end
 
-        sql = <<~SQL
+        <<~SQL
           INSERT INTO #{shadow_table}(#{insert_into_columns.join(", ")})
           SELECT #{select_columns.join(", ")}
           FROM ONLY #{client.table}

data/lib/pg_online_schema_change/replay.rb

@@ -1,3 +1,7 @@
+# frozen_string_literal: true
+
+# rubocop:disable Metrics/AbcSize, Metrics/MethodLength
+
 module PgOnlineSchemaChange
   class Replay
     extend Helper
@@ -5,7 +9,6 @@ module PgOnlineSchemaChange
     class << self
       PULL_BATCH_COUNT = 1000
       DELTA_COUNT = 20
-      RESERVED_COLUMNS = %w[operation_type trigger_time].freeze
 
       # This, picks PULL_BATCH_COUNT rows by primary key from audit_table,
       # replays it on the shadow_table. Once the batch is done,
@@ -25,7 +28,7 @@ module PgOnlineSchemaChange
 
       def rows_to_play(reuse_trasaction = false)
         select_query = <<~SQL
-          SELECT * FROM #{audit_table} ORDER BY #{primary_key} LIMIT #{PULL_BATCH_COUNT};
+          SELECT * FROM #{audit_table} ORDER BY #{audit_table_pk} LIMIT #{PULL_BATCH_COUNT};
         SQL
 
         rows = []
@@ -34,6 +37,10 @@ module PgOnlineSchemaChange
         rows
       end
 
+      def reserved_columns
+        @reserved_columns ||= [trigger_time_column, operation_type_column, audit_table_pk]
+      end
+
       def play!(rows, reuse_trasaction = false)
         logger.info("Replaying rows, count: #{rows.size}")
 
@@ -44,7 +51,7 @@ module PgOnlineSchemaChange
 
           # Remove audit table cols, since we will be
           # re-mapping them for inserts and updates
-          RESERVED_COLUMNS.each do |col|
+          reserved_columns.each do |col|
             new_row.delete(col)
           end
 
@@ -73,7 +80,7 @@ module PgOnlineSchemaChange
             client.connection.escape_string(value)
           end
 
-          case row["operation_type"]
+          case row[operation_type_column]
           when "INSERT"
             values = new_row.map { |_, val| "'#{val}'" }.join(",")
 
@@ -110,13 +117,15 @@ module PgOnlineSchemaChange
         Query.run(client.connection, to_be_replayed.join, reuse_trasaction)
 
         # Delete items from the audit now that are replayed
-        if to_be_deleted_rows.count >= 1
-          delete_query = <<~SQL
-            DELETE FROM #{audit_table} WHERE #{primary_key} IN (#{to_be_deleted_rows.join(",")})
-          SQL
-          Query.run(client.connection, delete_query, reuse_trasaction)
-        end
+        return unless to_be_deleted_rows.count >= 1
+
+        delete_query = <<~SQL
+          DELETE FROM #{audit_table} WHERE #{primary_key} IN (#{to_be_deleted_rows.join(",")})
+        SQL
+        Query.run(client.connection, delete_query, reuse_trasaction)
       end
     end
   end
 end
+
+# rubocop:enable Metrics/AbcSize, Metrics/MethodLength

data/lib/pg_online_schema_change/store.rb

@@ -1,17 +1,21 @@
+# frozen_string_literal: true
+
 require "pg_query"
 require "pg"
 
 module PgOnlineSchemaChange
   class Store
     class << self
-      @@object = {}
+      @object = {}
 
       def get(key)
-        @@object[key.to_s] || @@object[key.to_sym]
+        @object ||= {}
+        @object[key.to_s] || @object[key.to_sym]
       end
 
       def set(key, value)
-        @@object[key.to_sym] = value
+        @object ||= {}
+        @object[key.to_sym] = value
       end
     end
   end

data/lib/pg_online_schema_change/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgOnlineSchemaChange
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/lib/pg_online_schema_change.rb

@@ -18,16 +18,12 @@ module PgOnlineSchemaChange
   class CountBelowDelta < StandardError; end
   class AccessExclusiveLockNotAcquired < StandardError; end
 
-  def self.logger=(verbose)
-    @@logger ||= begin
-        logger = Ougai::Logger.new($stdout)
-        logger.level = verbose ? Ougai::Logger::TRACE : Ougai::Logger::INFO
-        logger.with_fields = { version: PgOnlineSchemaChange::VERSION }
-        logger
-      end
-  end
-
-  def self.logger
-    @@logger
+  def self.logger(verbose: false)
+    @logger ||= begin
+      logger = Ougai::Logger.new($stdout)
+      logger.level = verbose ? Ougai::Logger::TRACE : Ougai::Logger::INFO
+      logger.with_fields = { version: PgOnlineSchemaChange::VERSION }
+      logger
+    end
   end
 end

data/scripts/release.sh

@@ -11,8 +11,11 @@ gem build pg_online_schema_change.gemspec
 echo "=== Pushing gem ===="
 gem push pg_online_schema_change-$VERSION.gem
 
+echo "=== Sleeping for 5s ===="
+sleep 5 
+
 echo "=== Building Image ===="
-docker build . --build-arg VERSION=$VERSION -t pg-osc
+docker build . --build-arg VERSION=$VERSION -t shayonj/pg-osc:$VERSION
 
 echo "=== Tagging Image ===="
 docker image tag shayonj/pg-osc:$VERSION shayonj/pg-osc:latest
@@ -20,3 +23,6 @@ docker image tag shayonj/pg-osc:$VERSION shayonj/pg-osc:latest
 echo "=== Pushing Image ===="
 docker push shayonj/pg-osc:$VERSION
 docker push shayonj/pg-osc:latest
+
+echo "=== Cleaning up ===="
+rm pg_online_schema_change-$VERSION.gem

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pg_online_schema_change
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Shayon Mukherjee
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-21 00:00:00.000000000 Z
+date: 2022-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ougai
@@ -191,6 +191,7 @@ extra_rdoc_files: []
 files:
 - ".rspec"
 - ".rubocop.yml"
+- ".rubocop_todo.yml"
 - ".ruby-version"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md