DEV: Drop plugin related index from core (#20930)

The migration of the index has since been moved into the
discourse-akismet plugin where it should have always been.
This commit is contained in:
Alan Guo Xiang Tan 2023-04-04 07:31:09 +08:00 committed by GitHub
parent 894586afa9
commit e54359c135
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 13 additions and 16 deletions

View File

@ -17,7 +17,6 @@ end
#
# Indexes
#
# idx_post_custom_fields_akismet (post_id) WHERE (((name)::text = 'AKISMET_STATE'::text) AND (value = 'needs_review'::text))
# index_post_custom_fields_on_name_and_value (name, "left"(value, 200))
# index_post_custom_fields_on_notice (post_id) UNIQUE WHERE ((name)::text = 'notice'::text)
# index_post_custom_fields_on_post_id (post_id) UNIQUE WHERE ((name)::text = 'missing uploads'::text)

View File

@ -1,15 +0,0 @@
# frozen_string_literal: true
# This is our current pattern for data migrations needed by plugins, we prefer to keep them in core
# so schema is tightly controlled, especially if we are amending tables owned by core
#
# this index makes looking up posts requiring review much faster (20ms on meta)
class AddPostCustomFieldsAkismetIndex < ActiveRecord::Migration[5.1]
def change
add_index :post_custom_fields,
[:post_id],
name: "idx_post_custom_fields_akismet",
where: "name = 'AKISMET_STATE' AND value = 'needs_review'"
end
end

View File

@ -0,0 +1,13 @@
# frozen_string_literal: true
# This plugin related index was added to core way back in 2018 but it should not have been added to core in the first place.
# The index has since been moved into the plugin itself.
class DropIdxPostCustomFieldsAkismet < ActiveRecord::Migration[7.0]
def up
execute "DROP INDEX IF EXISTS idx_post_custom_fields_akismet"
end
def down
raise ActiveRecord::IrreversibleMigration
end
end