From dd4ef69839c78e4220cd821bc3ad369eb6e41fa7 Mon Sep 17 00:00:00 2001 From: aschmitz Date: Fri, 22 Sep 2017 06:20:04 -0500 Subject: [PATCH] Hardcode IdToBigints migration columns (#5039) This addresses a comment during review: https://github.com/tootsuite/mastodon/pull/4801#discussion_r139337452 This means we'll need to make sure that all _id columns going forward are bigints, but that should happen automatically in most cases. --- db/migrate/20170918125918_ids_to_bigints.rb | 127 ++++++++++++++++++++ db/schema.rb | 122 +++++++++---------- 2 files changed, 188 insertions(+), 61 deletions(-) create mode 100644 db/migrate/20170918125918_ids_to_bigints.rb diff --git a/db/migrate/20170918125918_ids_to_bigints.rb b/db/migrate/20170918125918_ids_to_bigints.rb new file mode 100644 index 000000000..9c81ecc05 --- /dev/null +++ b/db/migrate/20170918125918_ids_to_bigints.rb @@ -0,0 +1,127 @@ +class IdsToBigints < ActiveRecord::Migration[5.1] + def up + change_column :account_domain_blocks, :account_id, :bigint + change_column :account_domain_blocks, :id, :bigint + change_column :accounts, :id, :bigint + change_column :blocks, :account_id, :bigint + change_column :blocks, :id, :bigint + change_column :blocks, :target_account_id, :bigint + change_column :conversation_mutes, :account_id, :bigint + change_column :conversation_mutes, :id, :bigint + change_column :deprecated_preview_cards, :id, :bigint + change_column :domain_blocks, :id, :bigint + change_column :favourites, :account_id, :bigint + change_column :favourites, :id, :bigint + change_column :favourites, :status_id, :bigint + change_column :follow_requests, :account_id, :bigint + change_column :follow_requests, :id, :bigint + change_column :follow_requests, :target_account_id, :bigint + change_column :follows, :account_id, :bigint + change_column :follows, :id, :bigint + change_column :follows, :target_account_id, :bigint + change_column :imports, :account_id, :bigint + change_column :imports, :id, :bigint + change_column :media_attachments, :account_id, :bigint + change_column :media_attachments, :id, :bigint + change_column :mentions, :account_id, :bigint + change_column :mentions, :id, :bigint + change_column :mutes, :account_id, :bigint + change_column :mutes, :id, :bigint + change_column :mutes, :target_account_id, :bigint + change_column :notifications, :account_id, :bigint + change_column :notifications, :from_account_id, :bigint + change_column :notifications, :id, :bigint + change_column :oauth_access_grants, :application_id, :bigint + change_column :oauth_access_grants, :id, :bigint + change_column :oauth_access_grants, :resource_owner_id, :bigint + change_column :oauth_access_tokens, :application_id, :bigint + change_column :oauth_access_tokens, :id, :bigint + change_column :oauth_access_tokens, :resource_owner_id, :bigint + change_column :oauth_applications, :id, :bigint + change_column :oauth_applications, :owner_id, :bigint + change_column :reports, :account_id, :bigint + change_column :reports, :action_taken_by_account_id, :bigint + change_column :reports, :id, :bigint + change_column :reports, :target_account_id, :bigint + change_column :session_activations, :access_token_id, :bigint + change_column :session_activations, :user_id, :bigint + change_column :session_activations, :web_push_subscription_id, :bigint + change_column :settings, :id, :bigint + change_column :settings, :thing_id, :bigint + change_column :statuses, :account_id, :bigint + change_column :statuses, :application_id, :bigint + change_column :statuses, :in_reply_to_account_id, :bigint + change_column :stream_entries, :account_id, :bigint + change_column :stream_entries, :id, :bigint + change_column :subscriptions, :account_id, :bigint + change_column :subscriptions, :id, :bigint + change_column :tags, :id, :bigint + change_column :users, :account_id, :bigint + change_column :users, :id, :bigint + change_column :web_settings, :id, :bigint + change_column :web_settings, :user_id, :bigint + end + + def down + change_column :account_domain_blocks, :account_id, :integer + change_column :account_domain_blocks, :id, :integer + change_column :accounts, :id, :integer + change_column :blocks, :account_id, :integer + change_column :blocks, :id, :integer + change_column :blocks, :target_account_id, :integer + change_column :conversation_mutes, :account_id, :integer + change_column :conversation_mutes, :id, :integer + change_column :deprecated_preview_cards, :id, :integer + change_column :domain_blocks, :id, :integer + change_column :favourites, :account_id, :integer + change_column :favourites, :id, :integer + change_column :favourites, :status_id, :integer + change_column :follow_requests, :account_id, :integer + change_column :follow_requests, :id, :integer + change_column :follow_requests, :target_account_id, :integer + change_column :follows, :account_id, :integer + change_column :follows, :id, :integer + change_column :follows, :target_account_id, :integer + change_column :imports, :account_id, :integer + change_column :imports, :id, :integer + change_column :media_attachments, :account_id, :integer + change_column :media_attachments, :id, :integer + change_column :mentions, :account_id, :integer + change_column :mentions, :id, :integer + change_column :mutes, :account_id, :integer + change_column :mutes, :id, :integer + change_column :mutes, :target_account_id, :integer + change_column :notifications, :account_id, :integer + change_column :notifications, :from_account_id, :integer + change_column :notifications, :id, :integer + change_column :oauth_access_grants, :application_id, :integer + change_column :oauth_access_grants, :id, :integer + change_column :oauth_access_grants, :resource_owner_id, :integer + change_column :oauth_access_tokens, :application_id, :integer + change_column :oauth_access_tokens, :id, :integer + change_column :oauth_access_tokens, :resource_owner_id, :integer + change_column :oauth_applications, :id, :integer + change_column :oauth_applications, :owner_id, :integer + change_column :reports, :account_id, :integer + change_column :reports, :action_taken_by_account_id, :integer + change_column :reports, :id, :integer + change_column :reports, :target_account_id, :integer + change_column :session_activations, :access_token_id, :integer + change_column :session_activations, :user_id, :integer + change_column :session_activations, :web_push_subscription_id, :integer + change_column :settings, :id, :integer + change_column :settings, :thing_id, :integer + change_column :statuses, :account_id, :integer + change_column :statuses, :application_id, :integer + change_column :statuses, :in_reply_to_account_id, :integer + change_column :stream_entries, :account_id, :integer + change_column :stream_entries, :id, :integer + change_column :subscriptions, :account_id, :integer + change_column :subscriptions, :id, :integer + change_column :tags, :id, :integer + change_column :users, :account_id, :integer + change_column :users, :id, :integer + change_column :web_settings, :id, :integer + change_column :web_settings, :user_id, :integer + end +end diff --git a/db/schema.rb b/db/schema.rb index 9f42d46dd..cf4edf9e5 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -10,20 +10,20 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20170917153509) do +ActiveRecord::Schema.define(version: 20170918125918) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" - create_table "account_domain_blocks", id: :serial, force: :cascade do |t| - t.integer "account_id" + create_table "account_domain_blocks", force: :cascade do |t| + t.bigint "account_id" t.string "domain" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "domain"], name: "index_account_domain_blocks_on_account_id_and_domain", unique: true end - create_table "accounts", id: :serial, force: :cascade do |t| + create_table "accounts", force: :cascade do |t| t.string "username", default: "", null: false t.string "domain" t.string "secret", default: "", null: false @@ -68,16 +68,16 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["username", "domain"], name: "index_accounts_on_username_and_domain", unique: true end - create_table "blocks", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "target_account_id", null: false + create_table "blocks", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "target_account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "target_account_id"], name: "index_blocks_on_account_id_and_target_account_id", unique: true end - create_table "conversation_mutes", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false + create_table "conversation_mutes", force: :cascade do |t| + t.bigint "account_id", null: false t.bigint "conversation_id", null: false t.index ["account_id", "conversation_id"], name: "index_conversation_mutes_on_account_id_and_conversation_id", unique: true end @@ -101,7 +101,7 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["shortcode", "domain"], name: "index_custom_emojis_on_shortcode_and_domain", unique: true end - create_table "domain_blocks", id: :serial, force: :cascade do |t| + create_table "domain_blocks", force: :cascade do |t| t.string "domain", default: "", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -110,9 +110,9 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["domain"], name: "index_domain_blocks_on_domain", unique: true end - create_table "favourites", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "status_id", null: false + create_table "favourites", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "status_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "id"], name: "index_favourites_on_account_id_and_id" @@ -120,24 +120,24 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["status_id"], name: "index_favourites_on_status_id" end - create_table "follow_requests", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "target_account_id", null: false + create_table "follow_requests", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "target_account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "target_account_id"], name: "index_follow_requests_on_account_id_and_target_account_id", unique: true end - create_table "follows", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "target_account_id", null: false + create_table "follows", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "target_account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "target_account_id"], name: "index_follows_on_account_id_and_target_account_id", unique: true end - create_table "imports", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false + create_table "imports", force: :cascade do |t| + t.bigint "account_id", null: false t.integer "type", null: false t.boolean "approved", default: false, null: false t.datetime "created_at", null: false @@ -148,14 +148,14 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.datetime "data_updated_at" end - create_table "media_attachments", id: :serial, force: :cascade do |t| + create_table "media_attachments", force: :cascade do |t| t.bigint "status_id" t.string "file_file_name" t.string "file_content_type" t.integer "file_file_size" t.datetime "file_updated_at" t.string "remote_url", default: "", null: false - t.integer "account_id" + t.bigint "account_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "shortcode" @@ -166,8 +166,8 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["status_id"], name: "index_media_attachments_on_status_id" end - create_table "mentions", id: :serial, force: :cascade do |t| - t.integer "account_id" + create_table "mentions", force: :cascade do |t| + t.bigint "account_id" t.bigint "status_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -175,29 +175,29 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["status_id"], name: "index_mentions_on_status_id" end - create_table "mutes", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "target_account_id", null: false + create_table "mutes", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "target_account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["account_id", "target_account_id"], name: "index_mutes_on_account_id_and_target_account_id", unique: true end - create_table "notifications", id: :serial, force: :cascade do |t| - t.integer "account_id" + create_table "notifications", force: :cascade do |t| + t.bigint "account_id" t.bigint "activity_id" t.string "activity_type" t.datetime "created_at", null: false t.datetime "updated_at", null: false - t.integer "from_account_id" + t.bigint "from_account_id" t.index ["account_id", "activity_id", "activity_type"], name: "account_activity", unique: true t.index ["activity_id", "activity_type"], name: "index_notifications_on_activity_id_and_activity_type" t.index ["id", "account_id", "activity_type"], name: "index_notifications_on_id_and_account_id_and_activity_type", order: { id: :desc } end - create_table "oauth_access_grants", id: :serial, force: :cascade do |t| - t.integer "resource_owner_id", null: false - t.integer "application_id", null: false + create_table "oauth_access_grants", force: :cascade do |t| + t.bigint "resource_owner_id", null: false + t.bigint "application_id", null: false t.string "token", null: false t.integer "expires_in", null: false t.text "redirect_uri", null: false @@ -207,9 +207,9 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["token"], name: "index_oauth_access_grants_on_token", unique: true end - create_table "oauth_access_tokens", id: :serial, force: :cascade do |t| - t.integer "resource_owner_id" - t.integer "application_id" + create_table "oauth_access_tokens", force: :cascade do |t| + t.bigint "resource_owner_id" + t.bigint "application_id" t.string "token", null: false t.string "refresh_token" t.integer "expires_in" @@ -221,7 +221,7 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["token"], name: "index_oauth_access_tokens_on_token", unique: true end - create_table "oauth_applications", id: :serial, force: :cascade do |t| + create_table "oauth_applications", force: :cascade do |t| t.string "name", null: false t.string "uid", null: false t.string "secret", null: false @@ -231,7 +231,7 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.datetime "updated_at" t.boolean "superapp", default: false, null: false t.string "website" - t.integer "owner_id" + t.bigint "owner_id" t.string "owner_type" t.index ["owner_id", "owner_type"], name: "index_oauth_applications_on_owner_id_and_owner_type" t.index ["uid"], name: "index_oauth_applications_on_uid", unique: true @@ -264,37 +264,37 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["status_id", "preview_card_id"], name: "index_preview_cards_statuses_on_status_id_and_preview_card_id" end - create_table "reports", id: :serial, force: :cascade do |t| - t.integer "account_id", null: false - t.integer "target_account_id", null: false + create_table "reports", force: :cascade do |t| + t.bigint "account_id", null: false + t.bigint "target_account_id", null: false t.bigint "status_ids", default: [], null: false, array: true t.text "comment", default: "", null: false t.boolean "action_taken", default: false, null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false - t.integer "action_taken_by_account_id" + t.bigint "action_taken_by_account_id" t.index ["account_id"], name: "index_reports_on_account_id" t.index ["target_account_id"], name: "index_reports_on_target_account_id" end create_table "session_activations", force: :cascade do |t| - t.integer "user_id", null: false + t.bigint "user_id", null: false t.string "session_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "user_agent", default: "", null: false t.inet "ip" - t.integer "access_token_id" - t.integer "web_push_subscription_id" + t.bigint "access_token_id" + t.bigint "web_push_subscription_id" t.index ["session_id"], name: "index_session_activations_on_session_id", unique: true t.index ["user_id"], name: "index_session_activations_on_user_id" end - create_table "settings", id: :serial, force: :cascade do |t| + create_table "settings", force: :cascade do |t| t.string "var", null: false t.text "value" t.string "thing_type" - t.integer "thing_id" + t.bigint "thing_id" t.datetime "created_at" t.datetime "updated_at" t.index ["thing_type", "thing_id", "var"], name: "index_settings_on_thing_type_and_thing_id_and_var", unique: true @@ -322,7 +322,7 @@ ActiveRecord::Schema.define(version: 20170917153509) do create_table "statuses", force: :cascade do |t| t.string "uri" - t.integer "account_id", null: false + t.bigint "account_id", null: false t.text "text", default: "", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -331,8 +331,8 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.string "url" t.boolean "sensitive", default: false, null: false t.integer "visibility", default: 0, null: false - t.integer "in_reply_to_account_id" - t.integer "application_id" + t.bigint "in_reply_to_account_id" + t.bigint "application_id" t.text "spoiler_text", default: "", null: false t.boolean "reply", default: false, null: false t.integer "favourites_count", default: 0, null: false @@ -349,13 +349,13 @@ ActiveRecord::Schema.define(version: 20170917153509) do create_table "statuses_tags", id: false, force: :cascade do |t| t.bigint "status_id", null: false - t.integer "tag_id", null: false + t.bigint "tag_id", null: false t.index ["status_id"], name: "index_statuses_tags_on_status_id" t.index ["tag_id", "status_id"], name: "index_statuses_tags_on_tag_id_and_status_id", unique: true end - create_table "stream_entries", id: :serial, force: :cascade do |t| - t.integer "account_id" + create_table "stream_entries", force: :cascade do |t| + t.bigint "account_id" t.bigint "activity_id" t.string "activity_type" t.datetime "created_at", null: false @@ -365,12 +365,12 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["activity_id", "activity_type"], name: "index_stream_entries_on_activity_id_and_activity_type" end - create_table "subscriptions", id: :serial, force: :cascade do |t| + create_table "subscriptions", force: :cascade do |t| t.string "callback_url", default: "", null: false t.string "secret" t.datetime "expires_at" t.boolean "confirmed", default: false, null: false - t.integer "account_id", null: false + t.bigint "account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.datetime "last_successful_delivery_at" @@ -378,7 +378,7 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["account_id", "callback_url"], name: "index_subscriptions_on_account_id_and_callback_url", unique: true end - create_table "tags", id: :serial, force: :cascade do |t| + create_table "tags", force: :cascade do |t| t.string "name", default: "", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -386,9 +386,9 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.index ["name"], name: "index_tags_on_name", unique: true end - create_table "users", id: :serial, force: :cascade do |t| + create_table "users", force: :cascade do |t| t.string "email", default: "", null: false - t.integer "account_id", null: false + t.bigint "account_id", null: false t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "encrypted_password", default: "", null: false @@ -430,8 +430,8 @@ ActiveRecord::Schema.define(version: 20170917153509) do t.datetime "updated_at", null: false end - create_table "web_settings", id: :serial, force: :cascade do |t| - t.integer "user_id" + create_table "web_settings", force: :cascade do |t| + t.bigint "user_id" t.json "data" t.datetime "created_at", null: false t.datetime "updated_at", null: false