diff options
Diffstat (limited to 'lib/gitlab/background_migration/backfill_group_features.rb')
-rw-r--r-- | lib/gitlab/background_migration/backfill_group_features.rb | 31 |
1 files changed, 8 insertions, 23 deletions
diff --git a/lib/gitlab/background_migration/backfill_group_features.rb b/lib/gitlab/background_migration/backfill_group_features.rb index 084c788c8cb..4c3af7be319 100644 --- a/lib/gitlab/background_migration/backfill_group_features.rb +++ b/lib/gitlab/background_migration/backfill_group_features.rb @@ -3,34 +3,19 @@ module Gitlab module BackgroundMigration # Backfill group_features for an array of groups - class BackfillGroupFeatures < ::Gitlab::BackgroundMigration::BaseJob - include Gitlab::Database::DynamicModelHelpers - - def perform(start_id, end_id, batch_table, batch_column, sub_batch_size, pause_ms, batch_size) - pause_ms = 0 if pause_ms < 0 - - parent_batch_relation = relation_scoped_to_range(batch_table, batch_column, start_id, end_id) - parent_batch_relation.each_batch(column: batch_column, of: sub_batch_size, order_hint: :type) do |sub_batch| - batch_metrics.time_operation(:upsert_group_features) do - upsert_group_features(sub_batch, batch_size) - end - - sleep(pause_ms * 0.001) + class BackfillGroupFeatures < ::Gitlab::BackgroundMigration::BatchedMigrationJob + def perform(batch_size) + each_sub_batch( + operation_name: :upsert_group_features, + batching_arguments: { order_hint: :type }, + batching_scope: ->(relation) { relation.where(type: 'Group') } + ) do |sub_batch| + upsert_group_features(sub_batch, batch_size) end end - def batch_metrics - @batch_metrics ||= Gitlab::Database::BackgroundMigration::BatchMetrics.new - end - private - def relation_scoped_to_range(source_table, source_key_column, start_id, stop_id) - define_batchable_model(source_table, connection: connection) - .where(source_key_column => start_id..stop_id) - .where(type: 'Group') - end - def upsert_group_features(relation, batch_size) connection.execute( <<~SQL |