Merge branch 'vij-sync-remove-ps-wiki-size' into 'master'

Remove wiki_size index on project statistics

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/158623



Merged-by: Adam Hegyi <ahegyi@gitlab.com>
Approved-by: Adam Hegyi <ahegyi@gitlab.com>
Reviewed-by: Mohamed Hamda <mhamda@gitlab.com>
Co-authored-by: Vijay Hawoldar <vhawoldar@gitlab.com>

db/post_migrate/20240708130252_remove_idx_project_statistics_wiki_size_and_project_id_sync.rb

+# frozen_string_literal: true
+
+class RemoveIdxProjectStatisticsWikiSizeAndProjectIdSync < Gitlab::Database::Migration[2.2]
+  milestone '17.3'
+  disable_ddl_transaction!
+
+  INDEX_NAME = 'index_project_statistics_on_wiki_size_and_project_id'
+  COLUMNS = %i[wiki_size project_id]
+
+  def up
+    return unless should_run?
+
+    remove_concurrent_index_by_name :project_statistics, name: INDEX_NAME
+  end
+
+  def down
+    return unless should_run?
+
+    add_concurrent_index :project_statistics, COLUMNS, name: INDEX_NAME
+  end
+
+  def should_run?
+    Gitlab.com_except_jh?
+  end
+end

db/schema_migrations/20240708130252

+4b1b5505a9becd13c4001b16f07a2fa2f0e331b97fed81a93b098ba82fc95715
\ No newline at end of file

spec/migrations/20240611122408_remove_project_statistics_wiki_size_and_project_id_index_spec.rb

@@ -4,7 +4,9 @@
 
 require_migration!
 
-RSpec.describe RemoveProjectStatisticsWikiSizeAndProjectIdIndex, feature_category: :consumables_cost_management do
+RSpec.describe RemoveProjectStatisticsWikiSizeAndProjectIdIndex,
+  feature_category: :consumables_cost_management,
+  schema: 20240611122408 do
   let(:migration) { described_class.new }
   let(:postgres_async_indexes) { table(:postgres_async_indexes) }
 

spec/migrations/20240708130252_remove_idx_project_statistics_wiki_size_and_project_id_sync_spec.rb

+# frozen_string_literal: true
+
+require 'spec_helper'
+
+require_migration!
+
+RSpec.describe RemoveIdxProjectStatisticsWikiSizeAndProjectIdSync,
+  feature_category: :consumables_cost_management,
+  schema: 20240708130252 do
+  let(:migration) { described_class.new }
+
+  describe '#up' do
+    it 'does nothing when not on gitlab.com' do
+      expect(migration).not_to receive(:remove_concurrent_index_by_name)
+
+      migration.up
+    end
+
+    it 'removes the index when on gitlab.com', :saas do
+      expect(migration).to receive(:remove_concurrent_index_by_name)
+
+      migration.up
+    end
+  end
+
+  describe '#down' do
+    it 'does nothing when not on gitlab.com' do
+      expect(migration).not_to receive(:add_concurrent_index)
+
+      migration.down
+    end
+
+    it 're-adds the index when on gitlab.com', :saas do
+      expect(migration).to receive(:add_concurrent_index)
+
+      migration.down
+    end
+  end
+end