darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-21 07:51:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

Improve query performance

Browse Source 
Adding a filter to `app.dao.notifications_dao.is_delivery_slow_for_providers` query to improve the performance. By added Notifications.notification_type = 'sms' to the query it will improve the performance some analyse shows 500ms improvement, which is a good thing especially when the query is run once a minute.
This commit is contained in:
Rebecca Law
2021-11-30 16:42:32 +00:00
parent ad313065bf
commit 101498ec84
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/dao/notifications_dao.py
View File

@@ -558,6 +558,7 @@ def is_delivery_slow_for_providers(
ProviderDetails ProviderDetails
).outerjoin( ).outerjoin(
Notification, and_( Notification, and_(
Notification.notification_type == SMS_TYPE,
Notification.sent_by == ProviderDetails.identifier, Notification.sent_by == ProviderDetails.identifier,
Notification.created_at >= created_at, Notification.created_at >= created_at,
Notification.sent_at.isnot(None), Notification.sent_at.isnot(None),