bulk_dependency_eraser 4.0.0 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e215e610f3c90e10cb386499bea48adc2a12a6cc7ce2a35466de12d5dc2dbb1d
-  data.tar.gz: 3c56acb8837e54c3387a6c2754dcab92e88402084f99198ab62c0ee1ea46d45b
+  metadata.gz: 16b8469147c79e15bbd947d75f9b9e7aa243f219be6796f516967a4f134609bd
+  data.tar.gz: a7a45c24562a67a1762d120caa9b4a14ab211594245383be4b21a70fa9f5a5e0
 SHA512:
-  metadata.gz: c1a1b05c9fd919c87a4947409296d65129f82817722d2de883b5f09e873b0851364348face62dc96a2251bb2aa8c8cba678798e0083681c7fa94f3ad0e1b6df6
-  data.tar.gz: 321cda6cc39a9ef3af66023aca8dbd385d46357ead943908d659880f968f2c7e1d5c9c8035d7f8c1c022b21e505aa25fc07be6a93296c73e73c67e0079061ff3
+  metadata.gz: 8ea34f3b1963b7c8d7720292a47edffb1351f18b6ecb6822e2965fe96244700044d9dcba8124cc088033d3114da0775b0a3be0c4636424af11c101e5e90fb686
+  data.tar.gz: fe4901ea4c7f99ef0e5228b94280f76b6102ef9686c2a8a6fa5fcb4da768fc0303e1da2929a1672bfea77ebcb0e42c9692c7774df21bf891d4e8054979d2c3ea

data/lib/bulk_dependency_eraser/builder.rb

@@ -52,10 +52,13 @@ module BulkDependencyEraser
       # Applied to reading queries
       # - 1st priority of scopes
       reading_proc_scopes_per_class_name: {},
+      # Using PG system column CTID can result in a 10x deletion speed increase
+      # - is used in combination with the primary key column to ensure CTID hasn't changed.
+      use_pg_system_column_ctid: false,
     }.freeze
 
     # write access so that these can be edited in-place by end-users who might need to manually adjust deletion order.
-    attr_accessor :deletion_list, :nullification_list
+    attr_accessor :deletion_list, :nullification_list, :deletion_list_with_additional_identifiers
     attr_reader :ignore_table_deletion_list, :ignore_table_nullification_list
     attr_reader :query_schema_parser
     attr_reader :current_klass_name
@@ -65,6 +68,15 @@ module BulkDependencyEraser
     def initialize query:, opts: {}
       @query = query
       @deletion_list  = {}
+      # CTID column values are stored here
+      # -   key: <klass_name (with or without suffixes)>
+      # - value: Hash
+      #   -   key: Individual record's ID
+      #   - value: Hash
+      #     -   key: Column Name
+      #     - value: plucked column value
+      # - these additional identifiers will be used when deleting the klasses
+      @deletion_list_with_additional_identifiers = {}
       @nullification_list = {}
 
       # For any ignored table results, they will be stored here
@@ -95,9 +107,13 @@ module BulkDependencyEraser
       # - prior approach was to use table_name.classify, but we can't trust that approach.
       opts_c.ignore_tables.each do |table_name|
         table_names_to_parsed_klass_names.dig(table_name)&.each do |klass_name|
-          klass_index = deletion_index_key(klass_name.constantize)
-          ignore_table_deletion_list[klass_index]      = deletion_list.delete(klass_index)      if deletion_list.key?(klass_index)
-          ignore_table_nullification_list[klass_index] = nullification_list.delete(klass_index) if nullification_list.key?(klass_index)
+          klass = klass_name.constantize
+          # Delete klasses from the deletion/nullification lists if they are from ignored tables.
+          # - Handles klass name indexes, with and without suffices.
+          deletion_index_keys(klass).each do |klass_index|
+            ignore_table_deletion_list[klass_index]      = deletion_list.delete(klass_index)      if deletion_list.key?(klass_index)
+            ignore_table_nullification_list[klass_index] = nullification_list.delete(klass_index) if nullification_list.key?(klass_index)
+          end
         end
       end
 
@@ -137,27 +153,44 @@ module BulkDependencyEraser
       end
     end
 
-    def pluck_from_query(query, column = :id)
+    # @param [ActiveRecord::Relation] query
+    # @param [Symbol | Array<Symbol>] column - primary_key must be the first or only element in the column.
+    # @return [
+    #   [Array<String | Int>] array of column values,
+    #   [Hash] hash of column/value pairings - additional identifiers.
+    # ]
+    def pluck_from_query(query, column = :id, skip_ctid_check: false)
+      if column.is_a?(Array)
+        columns = column
+      else
+        columns = [column]
+      end
+
+      # Only pluck CTID if we're plucking column :id (the only primary key supported)
+      if columns == [:id] && opts_c.use_pg_system_column_ctid && skip_ctid_check == false
+        columns << :ctid
+      end
+
       set_current_klass_name(query)
       # ordering shouldn't matter in these queries, and would slow it down
       # - we're ignoring default_scope ordering, but assoc-defined ordering would still take effect
       query = query.reorder('')
       query = custom_scope_for_query(query)
 
-      query_ids = []
+      query_results = []
       read_from_db do
         # If the query has a limit, then we don't want to clobber with batching.
         if batching_disabled? || !query.where({}).limit_value.nil?
           # query without batching
-          query_ids = query.pluck(column)
+          query_results = query.pluck(*columns)
         elsif opts_c.disable_batch_ordering || opts_c.disable_batch_ordering_for_klasses.include?(current_klass_name)
           # query with orderless batching
           offset = 0
           loop do
-            new_query_ids = query.offset(offset).limit(batch_size).pluck(column)
-            query_ids += new_query_ids
+            new_query_results = query.offset(offset).limit(batch_size).pluck(*columns)
+            query_results += new_query_results
 
-            break if new_query_ids.size < batch_size
+            break if new_query_results.size < batch_size
 
             # Move to the next batch
             offset += batch_size
@@ -165,12 +198,35 @@ module BulkDependencyEraser
         else
           # query with ordered batching
           query.in_batches(of: batch_size) do |subset_query|
-            query_ids += subset_query.pluck(column)
+            query_results += subset_query.pluck(*columns)
           end
         end
       end
 
-      return query_ids
+      # Early exit if we only plucked one column.
+      # - query_results would just be an array of IDs
+      return [query_results, nil] if columns.count == 1
+
+      transposed_results = query_results.transpose
+
+      query_primary_keys = transposed_results.shift || [] # could be an empty id set
+      columns.shift # shift of the primary key column
+
+      query_additional_identifiers = {}
+      results_per_additional_column = {}
+      columns.each do |column|
+        column_results = transposed_results.shift
+        results_per_additional_column[column] = column_results
+      end
+
+      query_primary_keys.each_with_index do |primary_key, plucked_index|
+        query_additional_identifiers[primary_key] ||= {}
+        results_per_additional_column.each do |column, column_results|
+          query_additional_identifiers[primary_key][column] = column_results[plucked_index]
+        end
+      end
+
+      return query_primary_keys, query_additional_identifiers
     end
 
     def batch_size
@@ -181,6 +237,32 @@ module BulkDependencyEraser
       opts_c.disable_read_batching.nil? ? opts_c.disable_batching : opts_c.disable_read_batching
     end
 
+    # No need for recursion here. Nullifying does not effect nested dependencies
+    def nullification_query_parser(query, nullfication_klass, klass_foreign_key, klass_foreign_type = nil)
+      # We're not destroying these assocs (just nullifying foreign_key columns) so we don't need to parse their dependencies.
+      # Setting skip_ctid_check: true because we currently don't support CTID with nullifications yet
+      assoc_ids, additional_identifiers_by_id = pluck_from_query(query, skip_ctid_check: true)
+      klass_name = nullfication_klass.name
+
+      # No assoc_ids, no need to add it to the nullification list
+      return if assoc_ids.none?
+
+      klass_foreign_key = klass_foreign_key.to_s
+      nullification_list[klass_name] ||= {}
+      nullification_list[klass_name][klass_foreign_key] ||= []
+      nullification_list[klass_name][klass_foreign_key] += assoc_ids
+      nullification_list[klass_name][klass_foreign_key].uniq!
+
+
+      # Also nullify the 'type' field, if the association is polymorphic
+      if klass_foreign_type
+        klass_foreign_type = klass_foreign_type.to_s
+        nullification_list[klass_name][klass_foreign_type] ||= []
+        nullification_list[klass_name][klass_foreign_type] += assoc_ids
+        nullification_list[klass_name][klass_foreign_type].uniq!
+      end
+    end
+
     def deletion_query_parser query, association_parent = nil
       # necessary for "ActiveRecord::Reflection::ThroughReflection" use-case
       # force_through_destroy_chains = options[:force_destroy_chain] || {}
@@ -225,24 +307,28 @@ module BulkDependencyEraser
       end
 
       # Pluck IDs of the current query
-      query_ids = pluck_from_query(query)
+      query_ids, additional_identifiers_by_id = pluck_from_query(query)
 
       klass_index = initialize_deletion_list_for_klass(klass)
 
       # prevent infinite recursion here.
       # - Remove any IDs that have been processed before
-      query_ids = remove_already_deletion_processed_ids(klass, query_ids)
+      query_ids, additional_identifiers_by_id = remove_already_deletion_processed_ids(klass, query_ids, additional_identifiers_by_id:)
 
       # If ids are nil, let's find that error
       if query_ids.none? #|| query_ids.nil?
         # quick cleanup, if turns out was an empty class
-        deletion_list.delete(klass_index) if deletion_list[klass_index].none?
+        if deletion_list[klass_index].none?
+          deletion_list.delete(klass_index)
+          deletion_list_with_additional_identifiers.delete(klass_index)
+        end
         return
       end
 
       # Use-case: We have more IDs to process
       # - can now safely add to the list, since we've prevented infinite recursion
       deletion_list[klass_index] += query_ids
+      deletion_list_with_additional_identifiers[klass_index].merge!(additional_identifiers_by_id)
 
       # ignore associations that aren't a dependent destroyable type
       destroy_associations = query.reflect_on_all_associations.select do |reflection|
@@ -378,25 +464,25 @@ module BulkDependencyEraser
       end
 
       # Look for manually specified keys in the assocation first
-      specified_primary_key = reflection.options[:primary_key]&.to_s
-      specified_foreign_key = reflection.options[:foreign_key]&.to_s
+      specified_primary_key = reflection.options[:primary_key]&.to_sym
+      specified_foreign_key = reflection.options[:foreign_key]&.to_sym
       # For polymorphic_associations
       specified_foreign_type = nil
 
       # handle foreign_key edge cases
       if specified_foreign_key.nil?
         if reflection.options[:as]
-          specified_foreign_type = "#{reflection.options[:as]}_type"
-          specified_foreign_key = "#{reflection.options[:as]}_id"
+          specified_foreign_type = "#{reflection.options[:as]}_type".to_sym
+          specified_foreign_key = "#{reflection.options[:as]}_id".to_sym
           # Only filtering by type here, the extra work for a poly assoc. We filter by IDs later
-          assoc_query = assoc_query.where({ specified_foreign_type.to_sym => parent_class.name })
+          assoc_query = assoc_query.where({ specified_foreign_type => parent_class.name })
         else
-          specified_foreign_key = parent_class.table_name.singularize + "_id"
+          specified_foreign_key = "#{parent_class.table_name.singularize}_id".to_sym
         end
       end
 
       # Check to see if foreign_key exists in association class's table
-      unless assoc_klass.column_names.include?(specified_foreign_key)
+      unless assoc_klass.column_names.include?(specified_foreign_key.to_s)
         report_error(
           "
           For #{parent_class.name}'s assoc '#{assoc_klass.name}': Could not determine the assoc's foreign key.
@@ -409,11 +495,11 @@ module BulkDependencyEraser
       # Build association query, based on parent class's primary key and the assoc's foreign key
       # - handle primary key edge cases
       # - The associations might not be using the primary_key of the klass table, but we can support that here.
-      if specified_primary_key && specified_primary_key&.to_s != 'id'
-        alt_primary_ids = pluck_from_query(query, specified_primary_key)
-        assoc_query = assoc_query.where(specified_foreign_key.to_sym => alt_primary_ids)
+      if specified_primary_key && specified_primary_key != :id
+        alt_primary_ids, _ = pluck_from_query(query, specified_primary_key)
+        assoc_query = assoc_query.where(specified_foreign_key => alt_primary_ids)
       else
-        assoc_query = assoc_query.where(specified_foreign_key.to_sym => query_ids)
+        assoc_query = assoc_query.where(specified_foreign_key => query_ids)
       end
 
       # remove any ordering or limits imposed on the association queries from the association definitions
@@ -427,25 +513,7 @@ module BulkDependencyEraser
           deletion_query_parser(assoc_query, parent_class)
         end
       elsif type == :nullify
-        # No need for recursion here.
-        # - we're not destroying these assocs (just nullifying foreign_key columns) so we don't need to parse their dependencies.
-        assoc_ids = pluck_from_query(assoc_query)
-
-        # No assoc_ids, no need to add it to the nullification list
-        return if assoc_ids.none?
-
-        nullification_list[assoc_klass_name] ||= {}
-        nullification_list[assoc_klass_name][specified_foreign_key] ||= []
-        nullification_list[assoc_klass_name][specified_foreign_key] += assoc_ids
-        nullification_list[assoc_klass_name][specified_foreign_key].uniq!
-
-
-        # Also nullify the 'type' field, if the association is polymorphic
-        if specified_foreign_type
-          nullification_list[assoc_klass_name][specified_foreign_type] ||= []
-          nullification_list[assoc_klass_name][specified_foreign_type] += assoc_ids
-          nullification_list[assoc_klass_name][specified_foreign_type].uniq!
-        end
+        nullification_query_parser(assoc_query, assoc_klass, specified_foreign_key, specified_foreign_type)
       else
         raise "invalid parsing type: #{type}"
       end
@@ -531,7 +599,7 @@ module BulkDependencyEraser
         return
       end
 
-      foreign_keys = pluck_from_query(query, specified_foreign_key)
+      foreign_keys, _ = pluck_from_query(query, specified_foreign_key, skip_ctid_check: true)
       assoc_query = assoc_query.where(
         specified_primary_key.to_sym => foreign_keys
       )
@@ -690,8 +758,14 @@ module BulkDependencyEraser
       opts_c.db_read_wrapper.call(block)
     end
 
-    def remove_already_deletion_processed_ids(klass, new_ids)
+    # @return [
+    #  [Array] array of IDs that haven't already been added to the deletion list
+    #  [Hash | Nil] the additional deletion query identifiers, minus the ones already processed
+    #    - will be nil if we're not plucking additional columns.
+    # ]
+    def remove_already_deletion_processed_ids(klass, new_ids, additional_identifiers_by_id: nil)
       already_processed_ids = []
+      additional_identifiers_by_id ||= {}
 
       if is_a_circular_dependency_klass?(klass)
         klass_keys = find_circular_dependency_deletion_keys(klass)
@@ -702,7 +776,10 @@ module BulkDependencyEraser
         already_processed_ids = deletion_list[klass.name]
       end
 
-      new_ids - already_processed_ids
+      return [
+        (new_ids - already_processed_ids),
+        additional_identifiers_by_id.except(*already_processed_ids)
+      ]
     end
 
     # Initializes deletion_list index
@@ -714,9 +791,11 @@ module BulkDependencyEraser
         raise "circular_index already existed for klass: #{klass.name}" if deletion_list.key?(klass_index)
 
         deletion_list[klass_index] = []
+        deletion_list_with_additional_identifiers[klass_index] = {}
       else
         # Not a circular dependency, define as normal
         deletion_list[klass_index] ||= []
+        deletion_list_with_additional_identifiers[klass_index] ||= {}
       end
 
       klass_index
@@ -734,6 +813,14 @@ module BulkDependencyEraser
       deletion_list.keys.select { |key| key.match?(regex) }
     end
 
+    # returns any current index keys, with or without suffix, in the deletion_list for the given klass param
+    def deletion_index_keys(klass)
+      klass_keys = []
+      klass_keys << klass.name if deletion_list.keys.include?(klass.name)
+      klass_keys += find_circular_dependency_deletion_keys(klass)
+      return klass_keys
+    end
+
     # If circular dependency, append a index suffix to the deletion hash key
     # - they will be deleted in highest index to lowest index order.
     def deletion_index_key(klass, increment_circular_index: false)

data/lib/bulk_dependency_eraser/deleter.rb

@@ -53,8 +53,9 @@ module BulkDependencyEraser
       deletion_proc_scopes_per_class_name: {},
     }.freeze
 
-    def initialize class_names_and_ids: {}, opts: {}
+    def initialize class_names_and_ids: {}, additional_identifiers_by_id: {}, opts: {}
       @class_names_and_ids = class_names_and_ids
+      @additional_identifiers_by_id = additional_identifiers_by_id
       super(opts:)
     end
 
@@ -68,19 +69,23 @@ module BulkDependencyEraser
         begin
           class_names_and_ids.keys.reverse.each do |class_name|
             current_class_name = class_name
+            additional_identifiers = additional_identifiers_by_id[class_name]
+            raise "invalid state! #{class_name} not found in 'additional_identifiers_by_id' list" if additional_identifiers.nil?
+
+            # Last in, First out
             ids = class_names_and_ids[class_name].reverse
             klass = constantize(class_name)
 
             if opts_c.enable_invalid_foreign_key_detection
               # delete with referential integrity
-              delete_by_klass_and_ids(klass, ids)
+              delete_by_klass_and_ids(klass, ids, additional_identifiers:)
             else
               # delete without referential integrity
               # Disable any ActiveRecord::InvalidForeignKey raised errors.
               # - src: https://stackoverflow.com/questions/41005849/rails-migrations-temporarily-ignore-foreign-key-constraint
               #        https://apidock.com/rails/ActiveRecord/ConnectionAdapters/AbstractAdapter/disable_referential_integrity
               ActiveRecord::Base.connection.disable_referential_integrity do
-                delete_by_klass_and_ids(klass, ids)
+                delete_by_klass_and_ids(klass, ids, additional_identifiers:)
               end
             end
           end
@@ -94,7 +99,7 @@ module BulkDependencyEraser
 
     protected
 
-    attr_reader :class_names_and_ids
+    attr_reader :class_names_and_ids, :additional_identifiers_by_id
 
     def custom_scope_for_query(query)
       klass = query.klass
@@ -113,29 +118,49 @@ module BulkDependencyEraser
       opts_c.disable_delete_batching.nil? ? opts_c.disable_batching : opts_c.disable_delete_batching
     end
 
-    def delete_by_klass_and_ids klass, ids
+    def delete_by_klass_and_ids(klass, ids, additional_identifiers:)
       puts "Deleting #{klass.name}'s IDs: #{ids}" if opts_c.verbose
       query = klass.unscoped
       query = custom_scope_for_query(query)
 
       if batching_disabled?
         puts "Deleting without batching" if opts_c.verbose
+        # Get column-names/keys of any additional identifer columns
+        detected_additional_identifier_columns = additional_identifiers.values.flat_map(&:keys).uniq
         delete_in_db do
-          deletion_result = query.where(id: ids).delete_all
+          deletion_query = query.where(id: ids)
+          # Apply any additional query identifiers (i.e. :ctid column)
+          detected_additional_identifier_columns.each do |column|
+            deletion_query = deletion_query.where(column => additional_identifiers.values.pluck(column))
+          end
+
+          # Perform Deletion
+          deletion_result = deletion_query.delete_all
           # Returning the following data in the event that the gem-implementer wants to insert their own db_delete_wrapper proc
           # and have access to these objects in their proc.
             # - query can give them access to the klass and table_name
-          [deletion_result, query, ids]
+          [deletion_result, query, ids, additional_identifiers]
         end
       else
         puts "Deleting with batching" if opts_c.verbose
         ids.each_slice(batch_size) do |ids_subset|
+          additional_identifiers_subset = additional_identifiers.slice(*ids_subset)
+          # Get column-names/keys of any additional identifer columns
+          detected_additional_identifier_columns = additional_identifiers_subset.values.flat_map(&:keys).uniq
           delete_in_db do
-            deletion_result = query.where(id: ids_subset).delete_all
+            deletion_query = query.where(id: ids_subset)
+            # Apply any additional query identifiers (i.e. :ctid column)
+            detected_additional_identifier_columns.each do |column|
+              deletion_query = deletion_query.where(column => additional_identifiers_subset.values.pluck(column))
+            end
+
+            # Perform Deletion
+            deletion_result = deletion_query.delete_all
+
             # Returning the following data in the event that the gem-implementer wants to insert their own db_delete_wrapper proc
             # and have access to these objects in their proc.
             # - query can give them access to the klass and table_name
-            [deletion_result, query, ids_subset]
+            [deletion_result, query, ids_subset, additional_identifiers_subset]
           end
         end
       end

data/lib/bulk_dependency_eraser/manager.rb

@@ -7,7 +7,7 @@ module BulkDependencyEraser
       verbose: false,
     }.freeze
 
-    delegate :nullification_list, :deletion_list, to: :dependency_builder
+    delegate :nullification_list, :deletion_list, :deletion_list_with_additional_identifiers, to: :dependency_builder
     delegate :ignore_table_deletion_list, :ignore_table_nullification_list, to: :dependency_builder
     delegate :circular_dependency_klasses, :flat_dependencies_per_klass, to: :dependency_builder
 
@@ -49,7 +49,11 @@ module BulkDependencyEraser
     end
 
     def delete!
-      @deleter  = BulkDependencyEraser::Deleter.new(class_names_and_ids: deletion_list, opts:)
+      @deleter  = BulkDependencyEraser::Deleter.new(
+        class_names_and_ids: deletion_list,
+        additional_identifiers_by_id: deletion_list_with_additional_identifiers,
+        opts:
+      )
       deleter_execution = deleter.execute
       unless deleter_execution
         puts "Deleter execution FAILED" if opts_c.verbose

data/lib/bulk_dependency_eraser/version.rb

@@ -1,3 +1,3 @@
 module BulkDependencyEraser
-  VERSION = "4.0.0".freeze
+  VERSION = "4.1.0".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bulk_dependency_eraser
 version: !ruby/object:Gem::Version
-  version: 4.0.0
+  version: 4.1.0
 platform: ruby
 authors:
 - benjamin.dana.software.dev@gmail.com