discourse
/
discourse-topic-voting
mirror of https://github.com/discourse/discourse-topic-voting.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Be a bit smarter about migrations. 

CREATE TABLE LIKE INCLUDING ALL actually creates an index as well, so
there is no need to create an index again.

It is probably also better to set the nextval of a seq than to change
ownership from one table to another.
This commit is contained in:
Nat 2024-07-17 16:44:26 +08:00
parent a281dad418
commit 9beaf240dd
No known key found for this signature in database
GPG Key ID: 4938B35D927EC773
1 changed files with 9 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
db/migrate/20240707170311_rename_voting_tables.rb
View File

@ -18,13 +18,9 @@ class RenameVotingTables < ActiveRecord::Migration[7.0]
SQL
execute <<~SQL
ALTER SEQUENCE discourse_voting_topic_vote_count_id_seq
RENAME TO topic_voting_topic_vote_count_id_seq
SQL
execute <<~SQL
ALTER SEQUENCE topic_voting_topic_vote_count_id_seq
OWNED BY topic_voting_topic_vote_count.id
ALTER TABLE topic_voting_topic_vote_count
ALTER COLUMN id
SET DEFAULT nextval('discourse_voting_topic_vote_count_id_seq')
SQL
add_index :topic_voting_topic_vote_count, :topic_id, unique: true
@ -44,16 +40,10 @@ class RenameVotingTables < ActiveRecord::Migration[7.0]
SQL
execute <<~SQL
ALTER SEQUENCE discourse_voting_votes_id_seq
RENAME TO topic_voting_votes_id_seq
ALTER TABLE topic_voting_votes
ALTER COLUMN id
SET DEFAULT nextval('discourse_voting_votes_id_seq')
SQL
execute <<~SQL
ALTER SEQUENCE topic_voting_votes_id_seq
OWNED BY topic_voting_votes.id
SQL
add_index :topic_voting_votes, %i[user_id topic_id], unique: true
end
unless table_exists?(:topic_voting_category_settings)
@ -70,16 +60,10 @@ class RenameVotingTables < ActiveRecord::Migration[7.0]
SQL
execute <<~SQL
ALTER SEQUENCE discourse_voting_category_settings_id_seq
RENAME TO topic_voting_category_settings_id_seq
ALTER TABLE topic_voting_category_settings
ALTER COLUMN id
SET DEFAULT nextval('discourse_voting_category_settings_id_seq')
SQL
execute <<~SQL
ALTER SEQUENCE topic_voting_category_settings_id_seq
OWNED BY topic_voting_category_settings.id
SQL
add_index :topic_voting_category_settings, :category_id, unique: true
end
end