Patch #25022
Add an index on attachments.disk_filename
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | % Done: | 0% | ||
Category: | Performance | |||
Target version: | 3.4.0 |
Description
In a system with lots of attachments deleting one attachment can take very long due to the DB lookup in Attachment#delete_from_disk
which checks if the same physical file is used by any other attachment.
We had this issue at Planio in a setup with more than 500k attachments, where deleting a single one could take up to half a second. This patch adds an index on attachments.disk_filename
that fixed the issue for us.
Associated revisions
Adds an index to attachments.disk_filename (#25022).
Patch by Jens Kraemer.
History
#1
Updated by Go MAEDA over 5 years ago
- Category set to Performance
- Target version set to 3.4.0
Thanks for sharing the experience and the patch.
Setting target version to 3.4.0.
Attachment#delete_from_disk
: source:tags/3.3.2/app/models/attachment.rb#L145
#2
Updated by Jean-Philippe Lang about 5 years ago
- Subject changed from add an index on attachments.disk_filename to Add an index on attachments.disk_filename
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
That should be usefull indeed, thanks.