darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-04 10:21:14 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #897 from alphagov/fix-slow-delete

Browse Source 
remove slow join from template-last-use query
This commit is contained in:
Leo Hemsted
2017-04-11 16:03:21 +01:00
committed by GitHub
parent 1e510487e2 c7c47a029f
commit b55b1007d0
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/dao/notifications_dao.py
View File

@@ -124,10 +124,10 @@ def dao_get_template_usage(service_id, limit_days=None):
def dao_get_last_template_usage(template_id): def dao_get_last_template_usage(template_id):
return NotificationHistory.query.filter( return NotificationHistory.query.filter(
NotificationHistory.template_id == template_id, NotificationHistory.template_id == template_id,
NotificationHistory.key_type != KEY_TYPE_TEST) \ NotificationHistory.key_type != KEY_TYPE_TEST
.join(Template) \ ).order_by(
.order_by(desc(NotificationHistory.created_at)) \ desc(NotificationHistory.created_at)
.first() ).first()
@statsd(namespace="dao") @statsd(namespace="dao")