Change int primary keys to bigint (#1088)

Integer primary keys can cause problems over time on big tables, and in general it's just better practice to use bigints instead. Some of our larger tables use UUIDs instead of ints, so this only upgrades the tables that used ints. (Practically speaking, none of these tables are really *likely* to overflow their primary key sequence, but better safe than sorry here, especially with technical maintainers offboarding.)

Fixes #1067.

db/migrate/20230209200733_change_int_to_bigint_for_primary_keys.rb

@@ -0,0 +1,32 @@
+class ChangeIntToBigintForPrimaryKeys < ActiveRecord::Migration[7.0]
+  def change_serial_primary_key_type(table, key, key_type)
+    change_column table, key, key_type
+    execute "ALTER SEQUENCE #{table}_#{key}_seq AS #{key_type};"
+  end
+
+  def up
+    # Foreign Keys
+    change_column :annotations, :author_id, :bigint
+    change_column :imports, :user_id, :bigint
+    change_column :invitations, :issuer_id, :bigint
+    change_column :invitations, :redeemer_id, :bigint
+
+    # Primary Keys
+    change_serial_primary_key_type :imports, :id, :bigint
+    change_serial_primary_key_type :invitations, :id, :bigint
+    change_serial_primary_key_type :users, :id, :bigint
+  end
+
+  def down
+    # Primary Keys
+    change_serial_primary_key_type :imports, :id, :int
+    change_serial_primary_key_type :invitations, :id, :int
+    change_serial_primary_key_type :users, :id, :int
+
+    # Foreign Keys
+    change_column :annotations, :author_id, :int
+    change_column :imports, :user_id, :int
+    change_column :invitations, :issuer_id, :int
+    change_column :invitations, :redeemer_id, :int
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2023_01_24_184531) do
+ActiveRecord::Schema[7.0].define(version: 2023_02_09_200733) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "citext"
   enable_extension "pgcrypto"
@@ -19,7 +19,7 @@
 
   create_table "annotations", primary_key: "uuid", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.uuid "change_uuid", null: false
-    t.integer "author_id", null: false
+    t.bigint "author_id", null: false
     t.jsonb "annotation", null: false
     t.datetime "created_at", precision: nil, null: false
     t.datetime "updated_at", precision: nil, null: false
@@ -39,8 +39,8 @@
     t.index ["uuid_to"], name: "index_changes_on_uuid_to"
   end
 
-  create_table "imports", id: :serial, force: :cascade do |t|
-    t.integer "user_id"
+  create_table "imports", force: :cascade do |t|
+    t.bigint "user_id"
     t.integer "status", default: 0, null: false
     t.string "file"
     t.jsonb "processing_errors"
@@ -53,9 +53,9 @@
     t.index ["user_id"], name: "index_imports_on_user_id"
   end
 
-  create_table "invitations", id: :serial, force: :cascade do |t|
-    t.integer "issuer_id"
-    t.integer "redeemer_id"
+  create_table "invitations", force: :cascade do |t|
+    t.bigint "issuer_id"
+    t.bigint "redeemer_id"
     t.string "code"
     t.string "email"
     t.datetime "expires_on", precision: nil
@@ -132,7 +132,7 @@
     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.string "encrypted_password", default: "", null: false
     t.string "reset_password_token"