Merge pull request #3428 from ualbertalib/2826_db_performance

Improve performance of Active Storage lookups #2826
ualbertalib · Apr 22, 2024 · 9ed276d · 9ed276d
2 parents 22a9809 + e7246cc
commit 9ed276d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,11 @@ New entries in this file should aim to provide a meaningful amount of informatio
 
 ## [Unreleased]
 
+### Fixed
+* Improves performance of Active Storage lookups via adding an table index.
+  Requires a db:migration that creates the index concurrently thus does not
+  lock the DB table [PR#3428](https://github.com/ualbertalib/jupiter/pull/3428)
+
 ### Changed
 * #3420: Update bundler to match version in production [PR#3433](https://github.com/ualbertalib/jupiter/pull/3433)
 

diff --git a/db/migrate/20240327203605_active_storage_attachments.rb b/db/migrate/20240327203605_active_storage_attachments.rb
@@ -0,0 +1,6 @@
+class ActiveStorageAttachments < ActiveRecord::Migration[7.1]
+  disable_ddl_transaction!
+  def change
+    add_index :active_storage_attachments, :record_id, algorithm: :concurrently, if_not_exists: true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb